Skip to content

Fix environment variable merging to allow empty string values#415

Open
k44 wants to merge 1 commit into
AikidoSec:mainfrom
k44:fix/merge-safe-chain-proxy-env
Open

Fix environment variable merging to allow empty string values#415
k44 wants to merge 1 commit into
AikidoSec:mainfrom
k44:fix/merge-safe-chain-proxy-env

Conversation

@k44
Copy link
Copy Markdown

@k44 k44 commented Apr 15, 2026

This pull request fixes the logic in mergeSafeChainProxyEnvironmentVariables to ensure that environment variables with empty string values are correctly copied.
A corresponding test has been added to verify this behavior.

  • Fixes an issue where empty string values were previously ignored
  • Adds a unit test to cover this case

Summary by Aikido

Security Issues: 0 Quality Issues: 0 Resolved Issues: 0

🐛 Bugfixes

  • Allowed empty string environment variables to be preserved during merge

More info

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant