git://git.onelab.eu
/
sfa.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
trimmed useless imports, unstarred all imports
[sfa.git]
/
sfa
/
methods
/
remove_peer_object.py
diff --git
a/sfa/methods/remove_peer_object.py
b/sfa/methods/remove_peer_object.py
index
b9b337f
..
a510189
100644
(file)
--- a/
sfa/methods/remove_peer_object.py
+++ b/
sfa/methods/remove_peer_object.py
@@
-1,17
+1,12
@@
-from sfa.util.faults import
*
+from sfa.util.faults import
UnknownSfaType, SfaInvalidArgument
from sfa.util.method import Method
from sfa.util.parameter import Parameter, Mixed
from sfa.util.method import Method
from sfa.util.parameter import Parameter, Mixed
-from sfa.trust.auth import Auth
-from sfa.util.record import GeniRecord
-from sfa.util.genitable import GeniTable
-from sfa.util.debug import log
+from sfa.util.table import SfaTable
from sfa.trust.credential import Credential
from sfa.trust.credential import Credential
-from sfa.util.misc import *
-from types import StringTypes
class remove_peer_object(Method):
"""
class remove_peer_object(Method):
"""
- Remove a
n
peer object from the PLC records of a local aggregate.
+ Remove a peer object from the PLC records of a local aggregate.
This method will be called by registry.remove() while removing
a record from the local aggreage's PLCDB and sfa table. This
method need not be directly called by end-user.
This method will be called by registry.remove() while removing
a record from the local aggreage's PLCDB and sfa table. This
method need not be directly called by end-user.
@@
-26,31
+21,34
@@
class remove_peer_object(Method):
accepts = [
Parameter(str, "Credential string"),
accepts = [
Parameter(str, "Credential string"),
- Parameter(dict, "Record dictionary")
- Parameter(str, "Request hash"),
+ Parameter(dict, "Record dictionary"),
+ Mixed(Parameter(str, "Human readable name of the original caller"),
+ Parameter(None, "Origin hrn not specified"))
]
returns = Parameter(int, "1 if successful")
]
returns = Parameter(int, "1 if successful")
- def call(self, cred, record,
request_hash, caller_cred
=None):
- if caller_cred==None:
- caller_cred=cred
+ def call(self, cred, record,
origin_hrn
=None):
+ user_cred = Credential(string=cred)
+
#log the call
#log the call
- self.api.logger.info("interface: %s\tcaller-hrn: %s\ttarget-hrn: %s\tmethod-name: %s"%(self.api.interface, Credential(string=caller_cred).get_gid_caller().get_hrn(), record['hrn'], self.name))
- self.api.auth.authenticateCred(cred, [cred], request_hash)
+ if not origin_hrn:
+ origin_hrn = user_cred.get_gid_caller().get_hrn()
+ self.api.logger.info("interface: %s\tcaller-hrn: %s\ttarget-hrn: %s\tmethod-name: %s"%(self.api.interface, origin_hrn, record['hrn'], self.name))
+
self.api.auth.check(cred, "remove")
# Only allow the local interface or record owner to delete peer_records
try: self.api.auth.verify_object_permission(record['hrn'])
except: self.api.auth.verify_cred_is_me(cred)
self.api.auth.check(cred, "remove")
# Only allow the local interface or record owner to delete peer_records
try: self.api.auth.verify_object_permission(record['hrn'])
except: self.api.auth.verify_cred_is_me(cred)
- table =
Geni
Table()
+ table =
Sfa
Table()
hrn, type = record['hrn'], record['type']
records = table.find({'hrn': hrn, 'type': type })
for record in records:
hrn, type = record['hrn'], record['type']
records = table.find({'hrn': hrn, 'type': type })
for record in records:
- if record['peer_authority']:
- self.remove_plc_record(record)
- table.remove(record)
+
if record['peer_authority']:
+
self.remove_plc_record(record)
+
table.remove(record)
return 1
return 1
@@
-86,14
+84,14
@@
class remove_peer_object(Method):
self.api.plshell.DeleteSite(self.api.plauth, site['site_id'])
else:
self.api.plshell.DeleteSite(self.api.plauth, site['site_id'])
else:
- raise Unknown
Geni
Type(type)
+ raise Unknown
Sfa
Type(type)
return 1
def get_peer_name(self, peer_id):
peers = self.api.plshell.GetPeers(self.api.plauth, [peer_id], ['peername', 'shortname', 'hrn_root'])
if not peers:
return 1
def get_peer_name(self, peer_id):
peers = self.api.plshell.GetPeers(self.api.plauth, [peer_id], ['peername', 'shortname', 'hrn_root'])
if not peers:
- raise
Geni
InvalidArgument, "No such peer"
+ raise
Sfa
InvalidArgument, "No such peer"
peer = peers[0]
return peer['shortname']
peer = peers[0]
return peer['shortname']