summaryrefslogtreecommitdiffstats
path: root/ipalib/plugins/aci.py
diff options
context:
space:
mode:
authorPetr Viktorin <pviktori@redhat.com>2012-04-17 12:42:35 -0400
committerMartin Kosek <mkosek@redhat.com>2012-06-20 15:18:42 +0200
commit1235dfa7bf4b249eb6da8eab8d8a2c7b0eef98db (patch)
treea972262b4ebfdc10de052a05e94cb8961e827041 /ipalib/plugins/aci.py
parent1484ccc4049dc42a5a8a71713253894ade401573 (diff)
downloadfreeipa-1235dfa7bf4b249eb6da8eab8d8a2c7b0eef98db.tar.gz
freeipa-1235dfa7bf4b249eb6da8eab8d8a2c7b0eef98db.tar.xz
freeipa-1235dfa7bf4b249eb6da8eab8d8a2c7b0eef98db.zip
Fail on unknown Command options
When unknown keyword arguments are passed to a Command, raise an error instead of ignoring them. Options used when IPA calls its commands internally are listed in a new Command attribute called internal_options, and allowed. Previous patches (0b01751c, c45174d6, c5689e7f) made IPA not use unknown keyword arguments in its own commands and tests, but since that some violations were reintroduced in permission_find and tests. Fix those. Tests included; both a frontend unittest and a XML-RPC test via the ping plugin (which was untested previously). https://fedorahosted.org/freeipa/ticket/2509
Diffstat (limited to 'ipalib/plugins/aci.py')
-rw-r--r--ipalib/plugins/aci.py2
1 files changed, 2 insertions, 0 deletions
diff --git a/ipalib/plugins/aci.py b/ipalib/plugins/aci.py
index f7c6039a9..7a27ce116 100644
--- a/ipalib/plugins/aci.py
+++ b/ipalib/plugins/aci.py
@@ -610,6 +610,8 @@ class aci_mod(crud.Update):
takes_options = (_prefix_option,)
+ internal_options = ['rename']
+
msg_summary = _('Modified ACI "%(value)s"')
def execute(self, aciname, **kw):