diff options
author | Jan Cholasta <jcholast@redhat.com> | 2016-04-05 13:11:20 +0200 |
---|---|---|
committer | Jan Cholasta <jcholast@redhat.com> | 2016-05-25 16:06:26 +0200 |
commit | 278fa29906093ead4977ae68eb3bf37613329102 (patch) | |
tree | d78e8f455352feb29cf68b6840bd35bd0ecf2fa5 /ipalib/parameters.py | |
parent | 13b010685b60404e89ec0392ee3d4157fbb0788b (diff) | |
download | freeipa-278fa29906093ead4977ae68eb3bf37613329102.tar.gz freeipa-278fa29906093ead4977ae68eb3bf37613329102.tar.xz freeipa-278fa29906093ead4977ae68eb3bf37613329102.zip |
frontend: perform argument value validation only on server
Do not validate values of command arguments on the client and let the
server handle validation.
This will make the client more lightweight by not having it to carry
validation code and metadata with itself for the price of increasing
network traffic in case the validation fails.
Types of the arguments are still validated on both the client and the
server.
https://fedorahosted.org/freeipa/ticket/4739
Reviewed-By: David Kupka <dkupka@redhat.com>
Diffstat (limited to 'ipalib/parameters.py')
-rw-r--r-- | ipalib/parameters.py | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/ipalib/parameters.py b/ipalib/parameters.py index 1b35b396a..ccfc204d4 100644 --- a/ipalib/parameters.py +++ b/ipalib/parameters.py @@ -572,7 +572,6 @@ class Param(ReadOnly): value = self.get_default(**kw) else: value = self.convert(self.normalize(value)) - self.validate(value, supplied=self.name in kw) return value def get_param_name(self): |