Skip to content

util: fix parseEnv handling of invalid lines#57798

Merged
nodejs-github-bot merged 2 commits intonodejs:mainfrom
AugustinMauroy:util-fix-parseEnv-handling-of-invalid-lines
Apr 21, 2025
Merged

util: fix parseEnv handling of invalid lines#57798
nodejs-github-bot merged 2 commits intonodejs:mainfrom
AugustinMauroy:util-fix-parseEnv-handling-of-invalid-lines

Conversation

@AugustinMauroy
Copy link
Member

This PR fixes an issue with util.parseEnv() where invalid lines in the input were being incorrectly concatenated into key names instead of being skipped. The fix ensures that lines without an equals sign are properly skipped during parsing

Fixes: #56775

I lost my old pr due to a git failure 😅

#56778

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

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. c++ Issues and PRs that require attention from people who are familiar with C++. commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. config Issues or PRs related to the config subsystem needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

util.parseEnv creates keys from invalid, newline-separated lines

6 participants