binbashar / leverage / 35291107483

66%

master: 65%

Build Type

Pull #327

github

Committed by juanmatias

Commit Message

Fix | Cast MFA_ENABLED to a real boolean

`PathsHandler.mfa_enabled` stored the raw string from build.env
without casting it. Since any non-empty string (including "false")
is truthy in Python, `elif paths.mfa_enabled:` in
leverage/modules/auth.py always ran regardless of the actual value,
so MFA_ENABLED=false never disabled the MFA credential-refresh path.

This surfaced in CI as `refresh_layer_credentials_mfa` looking for a
`[profile <name>-mfa]` section that was never meant to exist (the
workflow explicitly sets MFA_ENABLED=false and sso_enabled=false to
skip credential refresh entirely for that layer), and raising
"Credentials for profile ... have not been properly configured."

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

Pull Request Pull Request #327: Fix | Credentials configure fails when run after project init

Coverage Stats

532 of 1040 branches covered (51.15%)

Branch coverage included in aggregate %.

29 of 32 new or added lines in 3 files covered. (90.63%)

93 existing lines in 6 files now uncovered.

2828 of 4048 relevant lines covered (69.86%)

0.7 hits per line