diff options
| author | Martin Kosek <mkosek@redhat.com> | 2012-01-20 15:15:11 +0100 |
|---|---|---|
| committer | Martin Kosek <mkosek@redhat.com> | 2012-01-20 16:28:09 +0100 |
| commit | ae2e49a22277a17269833ec13fa657fdc2547b36 (patch) | |
| tree | ba9d02e4126cea7d9dae03e074758bb373183f6d /ipalib/plugins | |
| parent | 092dd8db1293599a4b7f0ab33ea43e8082ec554f (diff) | |
| download | freeipa-ae2e49a22277a17269833ec13fa657fdc2547b36.tar.gz freeipa-ae2e49a22277a17269833ec13fa657fdc2547b36.tar.xz freeipa-ae2e49a22277a17269833ec13fa657fdc2547b36.zip | |
Improve host-add error message
host-add command allows to add a host and its IP address via
--ip-address option. When the address is invalid, it throws
an error and refuses to operate. However, the invalid IP address
error message is always the same which is not right as it forces
a user to guess the reason of rejection (loopback address,
link-local address or invalid address at all was passed, etc.).
This patch changes host-add validator to print the error message.
https://fedorahosted.org/freeipa/ticket/2229
Diffstat (limited to 'ipalib/plugins')
| -rw-r--r-- | ipalib/plugins/host.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ipalib/plugins/host.py b/ipalib/plugins/host.py index 33d60ad1c..fb5f4ea24 100644 --- a/ipalib/plugins/host.py +++ b/ipalib/plugins/host.py @@ -203,8 +203,8 @@ def validate_ipaddr(ugettext, ipaddr): """ try: ip = CheckedIPAddress(ipaddr, match_local=False) - except: - return _('invalid IP address') + except Exception, e: + return unicode(e) return None |