diff options
author | Martin Basti <mbasti@redhat.com> | 2014-03-20 09:47:20 +0100 |
---|---|---|
committer | Martin Kosek <mkosek@redhat.com> | 2014-06-03 15:55:32 +0200 |
commit | 0827b95d0a1949d9d942670bfc7a8cae91c59a16 (patch) | |
tree | 4fcaad40ded0eb9246df707e20d41f1e07726436 /ipalib | |
parent | dafd162ae5a9a5fa621c651b3a8aaf8ea174746e (diff) | |
download | freeipa-0827b95d0a1949d9d942670bfc7a8cae91c59a16.tar.gz freeipa-0827b95d0a1949d9d942670bfc7a8cae91c59a16.tar.xz freeipa-0827b95d0a1949d9d942670bfc7a8cae91c59a16.zip |
move hostname validation from DNS to hosts
Validator is no more used in dns plugin
Part of ticket:
IPA should allow internationalized domain names
https://fedorahosted.org/freeipa/ticket/3169
Reviewed-By: Jan Cholasta <jcholast@redhat.com>
Diffstat (limited to 'ipalib')
-rw-r--r-- | ipalib/plugins/host.py | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/ipalib/plugins/host.py b/ipalib/plugins/host.py index 64d018532..bd7a3a2ba 100644 --- a/ipalib/plugins/host.py +++ b/ipalib/plugins/host.py @@ -34,13 +34,12 @@ from ipalib.plugins.service import (split_principal, validate_certificate, set_kerberos_attrs) from ipalib.plugins.dns import (dns_container_exists, _record_types, add_records_for_host_validation, add_records_for_host, - _hostname_validator, get_reverse_zone) -from ipalib.plugins.dns import get_reverse_zone + get_reverse_zone) from ipalib import _, ngettext from ipalib import x509 from ipalib.request import context from ipalib.util import (normalize_sshpubkey, validate_sshpubkey_no_options, - convert_sshpubkey_post) + convert_sshpubkey_post, validate_hostname) from ipapython.ipautil import ipa_generate_password, CheckedIPAddress from ipapython.dnsutil import DNSName from ipapython.ssh import SSHPublicKey @@ -215,6 +214,15 @@ def normalize_hostname(hostname): hostname = hostname.lower() return hostname +def _hostname_validator(ugettext, value): + try: + validate_hostname(value) + except ValueError, e: + return _('invalid domain-name: %s') \ + % unicode(e) + + return None + class host(LDAPObject): """ Host object. |