]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
netcmd: silo member: rename exceptions to grant and revoke
authorRob van der Linde <rob@catalyst.net.nz>
Tue, 7 Nov 2023 19:30:22 +0000 (08:30 +1300)
committerAndrew Bartlett <abartlet@samba.org>
Wed, 15 Nov 2023 04:05:34 +0000 (04:05 +0000)
Signed-off-by: Rob van der Linde <rob@catalyst.net.nz>
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
python/samba/netcmd/domain/models/auth_silo.py
python/samba/netcmd/domain/models/exceptions.py

index 28d94e64fa3571b6a7d71219b26a7b1909c6e580..917e7dcafe6fc7391a5fa90187528ea02053a811 100644 (file)
@@ -24,7 +24,7 @@ from ldb import FLAG_MOD_ADD, FLAG_MOD_DELETE, LdbError, Message, MessageElement
 
 from samba.sd_utils import escaped_claim_id
 
-from .exceptions import AddMemberError, RemoveMemberError
+from .exceptions import GrantMemberError, RevokeMemberError
 from .fields import DnField, BooleanField, StringField
 from .model import Model
 
@@ -71,7 +71,7 @@ class AuthenticationSilo(Model):
         try:
             ldb.modify(message)
         except LdbError as e:
-            raise AddMemberError(f"Failed to add silo member: {e}")
+            raise GrantMemberError(f"Failed to add silo member: {e}")
 
         # If the modify operation was successful refresh members field.
         self.refresh(ldb, fields=["members"])
@@ -94,7 +94,7 @@ class AuthenticationSilo(Model):
         try:
             ldb.modify(message)
         except LdbError as e:
-            raise RemoveMemberError(f"Failed to remove silo member: {e}")
+            raise RevokeMemberError(f"Failed to remove silo member: {e}")
 
         # If the modify operation was successful refresh members field.
         self.refresh(ldb, fields=["members"])
index b28b423f64df6b964e5e86483851cba3e2eb9842..14ebd774d097fd353851e53d39ed47c8bc1584f4 100644 (file)
@@ -44,11 +44,11 @@ class DoesNotExist(ModelError):
     pass
 
 
-class AddMemberError(ModelError):
+class GrantMemberError(ModelError):
     pass
 
 
-class RemoveMemberError(ModelError):
+class RevokeMemberError(ModelError):
     pass