On 10/27/2015 07:16 PM, Martin Basti wrote:
On 27.10.2015 13:00, Abhijeet Kasurde wrote:
Hi All,
This patch fixes bug - https://fedorahosted.org/freeipa/ticket/4811
Thanks,
Abhijeet Kasurde
Sorry, my bad.
Please find the new patch.
[Tue Oct 27 14:44:51.328615 2015] [wsgi:error] [pid 5556] ipa: ERROR:
non-public: AttributeError: 'dnszone' object has no attribute
'handle_obj_found'
[Tue Oct 27 14:44:51.328654 2015] [wsgi:error] [pid 5556] Traceback
(most recent call last):
[Tue Oct 27 14:44:51.328659 2015] [wsgi:error] [pid 5556] File
"/usr/lib/python2.7/site-packages/ipaserver/rpcserver.py", line 350,
in wsgi_execute
[Tue Oct 27 14:44:51.328664 2015] [wsgi:error] [pid 5556] result =
self.Command[name](*args, **options)
[Tue Oct 27 14:44:51.328669 2015] [wsgi:error] [pid 5556] File
"/usr/lib/python2.7/site-packages/ipalib/frontend.py", line 447, in
__call__
[Tue Oct 27 14:44:51.328674 2015] [wsgi:error] [pid 5556] ret =
self.run(*args, **options)
[Tue Oct 27 14:44:51.328678 2015] [wsgi:error] [pid 5556] File
"/usr/lib/python2.7/site-packages/ipalib/frontend.py", line 764, in run
[Tue Oct 27 14:44:51.328683 2015] [wsgi:error] [pid 5556] return
self.execute(*args, **options)
[Tue Oct 27 14:44:51.328687 2015] [wsgi:error] [pid 5556] File
"/usr/lib/python2.7/site-packages/ipalib/plugins/dns.py", line 2935,
in execute
[Tue Oct 27 14:44:51.328692 2015] [wsgi:error] [pid 5556] result =
super(dnszone_enable, self).execute(*keys, **options)
[Tue Oct 27 14:44:51.328696 2015] [wsgi:error] [pid 5556] File
"/usr/lib/python2.7/site-packages/ipalib/plugins/dns.py", line 2262,
in execute
[Tue Oct 27 14:44:51.328701 2015] [wsgi:error] [pid 5556]
self.obj.handle_obj_found(*keys)
[Tue Oct 27 14:44:51.328705 2015] [wsgi:error] [pid 5556]
AttributeError: 'dnszone' object has no attribute 'handle_obj_found'
From 44dad99991e1dc4113970068a766ccca9c66edaa Mon Sep 17 00:00:00 2001
From: Abhijeet Kasurde <akasu...@redhat.com>
Date: Tue, 27 Oct 2015 17:21:17 +0530
Subject: [PATCH] Added user friendly error message for dnszone enable and
disable
Added try-except block in dns plugin in order to provide user
friendly message to end user.
https://fedorahosted.org/freeipa/ticket/4811
Signed-off-by: Abhijeet Kasurde <akasu...@redhat.com>
---
ipalib/plugins/dns.py | 12 ++++++++++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/ipalib/plugins/dns.py b/ipalib/plugins/dns.py
index ef282c94609df0be0aa02ec14eb2b137e7ad9dbd..48d6f740ebea06e0ae9e8d68deafd607b5ae18d8 100644
--- a/ipalib/plugins/dns.py
+++ b/ipalib/plugins/dns.py
@@ -2231,7 +2231,11 @@ class DNSZoneBase_disable(LDAPQuery):
ldap = self.obj.backend
dn = self.obj.get_dn(*keys, **options)
- entry = ldap.get_entry(dn, ['idnszoneactive', 'objectclass'])
+ try:
+ entry = ldap.get_entry(dn, ['idnszoneactive', 'objectclass'])
+ except errors.NotFound:
+ self.obj.handle_not_found(*keys)
+
if not _check_entry_objectclass(entry, self.obj.object_class):
self.obj.handle_not_found(*keys)
@@ -2252,7 +2256,11 @@ class DNSZoneBase_enable(LDAPQuery):
ldap = self.obj.backend
dn = self.obj.get_dn(*keys, **options)
- entry = ldap.get_entry(dn, ['idnszoneactive', 'objectclass'])
+ try:
+ entry = ldap.get_entry(dn, ['idnszoneactive', 'objectclass'])
+ except errors.NotFound:
+ self.obj.handle_not_found(*keys)
+
if not _check_entry_objectclass(entry, self.obj.object_class):
self.obj.handle_not_found(*keys)
--
2.4.3
--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code