Skip to content

Commit 1904ee4

Browse files
committed
Simplify logic
1 parent ccd39ff commit 1904ee4

1 file changed

Lines changed: 25 additions & 29 deletions

File tree

nxc/protocols/ldap.py

Lines changed: 25 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -767,35 +767,31 @@ def resolve_and_display_hostname(name, domain_name=None):
767767
prefix = f"[{domain_name}] " if domain_name else ""
768768
try:
769769
# Resolve using DNS server for A, AAAA, CNAME, PTR, and NS records
770-
if name:
771-
found_record = False
772-
for record_type in ["A", "AAAA", "CNAME", "PTR", "NS"]:
773-
if found_record: # Flag to check if any record is found
774-
break # If a record has been found, stop checking further
775-
try:
776-
answers = resolv.resolve(name, record_type, tcp=self.args.dns_tcp)
777-
for rdata in answers:
778-
if record_type in ["A", "AAAA"]:
779-
ip_address = rdata.to_text()
780-
self.logger.highlight(f"{prefix}{name} = {colored(ip_address, host_info_colors[0])}")
781-
found_record = True # Set flag to true since a record is found
782-
elif record_type == "CNAME":
783-
self.logger.highlight(f"{prefix}{name} CNAME = {colored(rdata.to_text(), host_info_colors[0])}")
784-
found_record = True
785-
elif record_type == "PTR":
786-
self.logger.highlight(f"{prefix}{name} PTR = {colored(rdata.to_text(), host_info_colors[0])}")
787-
found_record = True
788-
elif record_type == "NS":
789-
self.logger.highlight(f"{prefix}{name} NS = {colored(rdata.to_text(), host_info_colors[0])}")
790-
found_record = True
791-
except resolver.NXDOMAIN:
792-
self.logger.fail(f"{prefix}{name} = Host not found (NXDOMAIN)")
793-
except resolver.Timeout:
794-
self.logger.fail(f"{prefix}{name} = Connection timed out")
795-
except resolver.NoAnswer:
796-
self.logger.fail(f"{prefix}{name} = DNS server did not respond")
797-
except Exception as e:
798-
self.logger.fail(f"{prefix}{name} encountered an unexpected error: {e}")
770+
for record_type in ["A", "AAAA", "CNAME", "PTR", "NS"]:
771+
try:
772+
answers = resolv.resolve(name, record_type, tcp=self.args.dns_tcp)
773+
for rdata in answers:
774+
if record_type in ["A", "AAAA"]:
775+
ip_address = rdata.to_text()
776+
self.logger.highlight(f"{prefix}{name} = {colored(ip_address, host_info_colors[0])}")
777+
return
778+
elif record_type == "CNAME":
779+
self.logger.highlight(f"{prefix}{name} CNAME = {colored(rdata.to_text(), host_info_colors[0])}")
780+
return
781+
elif record_type == "PTR":
782+
self.logger.highlight(f"{prefix}{name} PTR = {colored(rdata.to_text(), host_info_colors[0])}")
783+
return
784+
elif record_type == "NS":
785+
self.logger.highlight(f"{prefix}{name} NS = {colored(rdata.to_text(), host_info_colors[0])}")
786+
return
787+
except resolver.NXDOMAIN:
788+
self.logger.fail(f"{prefix}{name} = Host not found (NXDOMAIN)")
789+
except resolver.Timeout:
790+
self.logger.fail(f"{prefix}{name} = Connection timed out")
791+
except resolver.NoAnswer:
792+
self.logger.fail(f"{prefix}{name} = DNS server did not respond")
793+
except Exception as e:
794+
self.logger.fail(f"{prefix}{name} encountered an unexpected error: {e}")
799795
else:
800796
self.logger.fail(f"{prefix} dNSHostName value is empty, unable to process.")
801797
except Exception as e:

0 commit comments

Comments
 (0)