mirror of
https://salsa.debian.org/freeipa-team/freeipa.git
synced 2025-02-25 18:55:28 -06:00
Docstring+refactor of IPADiscovery.ipadnssearchkrbrealm()
Added a docstring and made a tiny miny refactor to IPADiscovery.ipadnssearchkrbrealm() Reviewed-By: Martin Basti <mbasti@redhat.com>
This commit is contained in:
committed by
Martin Basti
parent
1cab1e980c
commit
c8cc62564b
@@ -514,7 +514,11 @@ class IPADiscovery(object):
|
|||||||
return servers
|
return servers
|
||||||
|
|
||||||
def ipadnssearchkrbrealm(self, domain=None):
|
def ipadnssearchkrbrealm(self, domain=None):
|
||||||
realm = None
|
"""
|
||||||
|
:param domain: Domain to be searched in
|
||||||
|
:returns: string of a realm found in a TXT record
|
||||||
|
None if no realm was found
|
||||||
|
"""
|
||||||
if not domain:
|
if not domain:
|
||||||
domain = self.domain
|
domain = self.domain
|
||||||
# now, check for a Kerberos realm the local host or domain is in
|
# now, check for a Kerberos realm the local host or domain is in
|
||||||
@@ -528,6 +532,7 @@ class IPADiscovery(object):
|
|||||||
root_logger.debug("DNS record not found: %s", e.__class__.__name__)
|
root_logger.debug("DNS record not found: %s", e.__class__.__name__)
|
||||||
answers = []
|
answers = []
|
||||||
|
|
||||||
|
realm = None
|
||||||
for answer in answers:
|
for answer in answers:
|
||||||
root_logger.debug("DNS record found: %s", answer)
|
root_logger.debug("DNS record found: %s", answer)
|
||||||
if answer.strings:
|
if answer.strings:
|
||||||
@@ -539,8 +544,7 @@ class IPADiscovery(object):
|
|||||||
.format(err=e))
|
.format(err=e))
|
||||||
continue
|
continue
|
||||||
if realm:
|
if realm:
|
||||||
break
|
return realm
|
||||||
return realm
|
|
||||||
|
|
||||||
def ipadnssearchkrbkdc(self, domain=None):
|
def ipadnssearchkrbkdc(self, domain=None):
|
||||||
kdc = None
|
kdc = None
|
||||||
|
|||||||
Reference in New Issue
Block a user