diff options
author | Petr Viktorin <pviktori@redhat.com> | 2012-05-21 05:03:21 -0400 |
---|---|---|
committer | Martin Kosek <mkosek@redhat.com> | 2012-05-29 09:23:26 +0200 |
commit | 1af36da933cd3c788e3a48257e2f5c286e985e22 (patch) | |
tree | 472816360fa7ad147e958b63e240f45ed04a72b0 /ipalib/plugable.py | |
parent | e0930d42a54e586a0170c853fbc9e66f9193d5b0 (diff) | |
download | freeipa-1af36da933cd3c788e3a48257e2f5c286e985e22.tar.gz freeipa-1af36da933cd3c788e3a48257e2f5c286e985e22.tar.xz freeipa-1af36da933cd3c788e3a48257e2f5c286e985e22.zip |
Disallow setattr on no_update/no_create params
Make --{set,add,del}attr fail on parameters with the no_update/no_create
flag for the respective command.
For attributes that can be modified, but we just don't want to display
in the CLI, use the 'no_option' flag. These are "locking" attributes
(ipaenabledflag, nsaccountlock) and externalhost.
Document the 'no_option' flag. Add some tests.
https://fedorahosted.org/freeipa/ticket/2580
Diffstat (limited to 'ipalib/plugable.py')
-rw-r--r-- | ipalib/plugable.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ipalib/plugable.py b/ipalib/plugable.py index 293db9241..f3d185e14 100644 --- a/ipalib/plugable.py +++ b/ipalib/plugable.py @@ -646,7 +646,7 @@ class API(DictProxy): if self.env.startup_traceback: import traceback self.log.error('could not load plugin module %r\n%s', pyfile, traceback.format_exc()) - raise e + raise def finalize(self): """ |