Skip to content

Commit

Permalink
FIX: env validation not logging all vars
Browse files Browse the repository at this point in the history
There was a bug that was causing us to not log required environment
variables. Now they will.
  • Loading branch information
mzappitello committed Jul 15, 2024
1 parent 4bb5365 commit 525b015
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/lamp_py/runtime_utils/env_validation.py
Original file line number Diff line number Diff line change
Expand Up @@ -45,10 +45,11 @@ def validate_environment(
value = os.environ.get(key, None)
if value is None:
missing_required.append(key)

# do not log private variables
if key in private_variables:
value = "**********"
metadata[key] = value
metadata[key] = value

# for optional variables, access ones that exist and add them to logs.
if optional_variables:
Expand Down

0 comments on commit 525b015

Please sign in to comment.