Skip to content

Commit 6497df2

Browse files
committed
Fix case sensitive environment variable lookup
1 parent fbbe8cd commit 6497df2

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

cloud_governance/main/environment_variables.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -392,17 +392,18 @@ def get_aws_account_alias_name(self):
392392
def get_env(var: str, defval=''):
393393
lcvar = var.lower()
394394
dashvar = lcvar.replace('_', '-')
395+
env_value = os.environ.get(var) or os.environ.get(var.upper()) or os.environ.get(var.lower()) or defval
395396
parser = argparse.ArgumentParser(description='Run CloudGovernance', allow_abbrev=False)
396397
if lcvar == dashvar:
397-
parser.add_argument(f"--{lcvar}", default=os.environ.get(var, defval), type=str, metavar='String', help=var)
398+
parser.add_argument(f"--{lcvar}", default=env_value, type=str, metavar='String', help=var)
398399
else:
399-
parser.add_argument(f"--{lcvar}", f"--{dashvar}", default=os.environ.get(var, defval), type=str,
400+
parser.add_argument(f"--{lcvar}", f"--{dashvar}", default=env_value, type=str,
400401
metavar='String', help=var)
401402
args, ignore = parser.parse_known_args()
402403
if hasattr(args, lcvar):
403404
return getattr(args, lcvar)
404405
else:
405-
return os.environ.get(var, defval)
406+
return env_value
406407

407408
@staticmethod
408409
def get_boolean_from_environment(var: str, defval: bool):

0 commit comments

Comments
 (0)