diff options
author | Martin Basti <mbasti@redhat.com> | 2016-09-20 13:00:53 +0200 |
---|---|---|
committer | Martin Babinsky <mbabinsk@redhat.com> | 2016-09-20 17:35:28 +0200 |
commit | 8f8e3d008f1de91337a83ea6d271662432209767 (patch) | |
tree | 05de56d6849ba5eebbecd6702cc81487922f21da /ipaserver/plugins/baseuser.py | |
parent | 37200806118d39ef8afe84ad5887a294d54e2659 (diff) | |
download | freeipa-8f8e3d008f1de91337a83ea6d271662432209767.tar.gz freeipa-8f8e3d008f1de91337a83ea6d271662432209767.tar.xz freeipa-8f8e3d008f1de91337a83ea6d271662432209767.zip |
Use constant for user and group patterns
User and groups regexp are the same and constant should be used to avoid
any future misconfigurations.
https://fedorahosted.org/freeipa/ticket/5822
Reviewed-By: Stanislav Laznicka <slaznick@redhat.com>
Reviewed-By: Alexander Bokovoy <abokovoy@redhat.com>
Diffstat (limited to 'ipaserver/plugins/baseuser.py')
-rw-r--r-- | ipaserver/plugins/baseuser.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/ipaserver/plugins/baseuser.py b/ipaserver/plugins/baseuser.py index 608e2d4fb..4c7e9f083 100644 --- a/ipaserver/plugins/baseuser.py +++ b/ipaserver/plugins/baseuser.py @@ -33,6 +33,7 @@ from ipaserver.plugins.service import ( validate_certificate, validate_realm, normalize_principal) from ipalib.request import context from ipalib import _ +from ipalib.constants import PATTERN_GROUPUSER_NAME from ipapython import kerberos from ipapython.ipautil import ipa_generate_password, GEN_TMP_PWD_LEN from ipapython.ipavalidate import Email @@ -172,7 +173,7 @@ class baseuser(LDAPObject): takes_params = ( Str('uid', - pattern='^[a-zA-Z0-9_.][a-zA-Z0-9_.-]*[a-zA-Z0-9_.$-]?$', + pattern=PATTERN_GROUPUSER_NAME, pattern_errmsg='may only include letters, numbers, _, -, . and $', maxlength=255, cli_name='login', |