Skip to content

Commit

Permalink
Modify check of public egress tag value
Browse files Browse the repository at this point in the history
This change aligns with the code that sets this tag in cisagov/cool-assessment-terraform (see cisagov/cool-assessment-terraform#240).
  • Loading branch information
dav3r committed May 15, 2024
1 parent bfb3849 commit 202aa79
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/lambda_handler.py
Original file line number Diff line number Diff line change
Expand Up @@ -94,8 +94,8 @@ def get_ec2_ips(
# Convert instance tags from an AWS dictionary into a Python dictionary
tags = convert_tags(instance)

# If the publish egress tag doesn't exist or isn't set to True, skip it
if tags.get(publish_egress_tag_name, str(False)) != str(True):
# If the publish egress tag doesn't exist or isn't set to true, skip it
if tags.get(publish_egress_tag_name, "false").lower() != "true":
continue
# Send back a tuple associating the public IP to an application.
# If application is unset, return "", so that the IP can be included
Expand All @@ -106,8 +106,8 @@ def get_ec2_ips(
# Convert elastic IP tags from an AWS dictionary into a Python dictionary
eip_tags = convert_tags(vpc_address)

# If the publish egress tag doesn't exist or isn't set to True, skip it
if eip_tags.get(publish_egress_tag_name, str(False)) != str(True):
# If the publish egress tag doesn't exist or isn't set to true, skip it
if eip_tags.get(publish_egress_tag_name, "false").lower() != "true":
continue
# Send back a tuple associating the public IP to an application.
# If application is unset, return "", so that the IP can be included
Expand Down

0 comments on commit 202aa79

Please sign in to comment.