You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
created a new def process_delete_record function to combine "repeating" tasks between A, PTR, and CNAME process. The code was combined with what was there before. Instead of doing the same process over 3 times, I just created a function and call it 3 times.
The def process_delete_record function performs:
checks TXT heritage validation (do I own it? does the instance ID match?)
Also sends SNS notification if heritage validation fails.
If not not pass validation, it skips deletion and sns notify
If so, it deletes the records - (A, PTR, or CNAME and associated TXT)
Fixed LOGGER.info lines that had a typo (missing parameter) in the sns notification caller. Example line 2274 but several others.
Fixed lOGGER.info to convert MAX_API_RETRY to string when writing to log. Example line 2264 but several others.
Updated new_get_resource_record with several fixes.
Return "" instead of Non if not found -- better align with parent function that calls it.
Checks the value of the list_resource_record_sets to match what should be returned. This is because the list_resource_record_sets API returns the next record (instead of matching exact) so wrong record can be retrieved if we do not check it.
MaxItem value changed to string.
Fixed def parse_heritage to add more logging and better input validation
Fixed def get_heritage_item to check input type and add more logging