From 6a523bba87812ecd274297a81a433b962fdd2e2a Mon Sep 17 00:00:00 2001 From: ducoterra Date: Sat, 1 Aug 2026 13:23:32 -0400 Subject: [PATCH] add better route53 logging --- route53_update.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/route53_update.py b/route53_update.py index 83adf55..2b86200 100644 --- a/route53_update.py +++ b/route53_update.py @@ -45,14 +45,18 @@ def _get_current_record(hosted_zone_id: str, record: str, record_type: str) -> s if rrset["Name"].rstrip(".") == record_normalized and rrset["Type"] == record_type: records = rrset.get("ResourceRecords", []) if records: - return records[0]["Value"].rstrip(".") + value = records[0]["Value"].rstrip(".") + logger.debug("Found %s record %s: value='%s' (len=%d)", record_type, record, value, len(value)) + return value except Exception as e: logger.warning("Failed to get current %s record for %s: %s", record_type, record, e) + logger.debug("No %s record found for %s", record_type, record) return None def update_ipv4(hosted_zone_id: str, record: str, public_ipv4: str) -> bool: current = _get_current_record(hosted_zone_id, record, "A") + logger.debug("IPv4 comparison for %s: current='%s' (len=%d) vs desired='%s' (len=%d)", record, current, len(current or ""), public_ipv4, len(public_ipv4)) if current == public_ipv4: logger.info("IPv4 for %s is already %s, no update needed", record, public_ipv4) return False @@ -87,6 +91,7 @@ def update_ipv4(hosted_zone_id: str, record: str, public_ipv4: str) -> bool: def update_ipv6(hosted_zone_id: str, record: str, public_ipv6: str) -> bool: current = _get_current_record(hosted_zone_id, record, "AAAA") + logger.debug("IPv6 comparison for %s: current='%s' (len=%d) vs desired='%s' (len=%d)", record, current, len(current or ""), public_ipv6, len(public_ipv6)) if current == public_ipv6: logger.info("IPv6 for %s is already %s, no update needed", record, public_ipv6) return False