diff options
| author | Petr Viktorin <pviktori@redhat.com> | 2012-02-24 12:26:28 -0500 |
|---|---|---|
| committer | Rob Crittenden <rcritten@redhat.com> | 2012-04-03 17:43:27 -0400 |
| commit | 8a7d7aaf810503befc174ce0465a08d086e47b91 (patch) | |
| tree | 93eafb7c3327be49c4c17b04eec69df3fdfc7b44 /ipaserver/install/httpinstance.py | |
| parent | a5a4323946c7131fcf646fd5bf4f7638e4ceca66 (diff) | |
| download | freeipa-8a7d7aaf810503befc174ce0465a08d086e47b91.tar.gz freeipa-8a7d7aaf810503befc174ce0465a08d086e47b91.tar.xz freeipa-8a7d7aaf810503befc174ce0465a08d086e47b91.zip | |
Defer conversion and validation until after --{add,del,set}attr are handled
--addattr & friends that modified attributes known to Python sometimes
used converted and validated Python values instead of LDAP strings.
This caused a problem for --delattr, which searched for a converted
integer in a list of raw strings (ticket 2407).
With this patch we work on raw strings, converting only when done.
Deferring validation ensures the end result is valid, so proper errors
are raised instead of failing later (ticket 2405).
Tests included.
Replaces previous fix for:
https://fedorahosted.org/freeipa/ticket/2418
Fixes:
https://fedorahosted.org/freeipa/ticket/2405
https://fedorahosted.org/freeipa/ticket/2407
https://fedorahosted.org/freeipa/ticket/2408
Diffstat (limited to 'ipaserver/install/httpinstance.py')
0 files changed, 0 insertions, 0 deletions