Skip to content

Conversation

desrosj
Copy link
Member

@desrosj desrosj commented Aug 19, 2025

This wraps all string values in double quotes (") to prevent word splitting.

This is being split off from #212 since this specific change was unrelated to accomplishing the goal of that pull request.

This also applies the change for all variables expecting a string type instead of just some.

This wraps all string values in double quotes (`"`) to prevent word splitting.

This is being split off from #212 since this specific change was unrelated to accomplishing the goal of that pull request.

This also applies the change for all variables expecting a string type instead of just some.

Co-Authored-By: Javier Casares <javier@casares.org>
@desrosj desrosj requested a review from javiercasares August 19, 2025 15:44
@desrosj desrosj self-assigned this Aug 19, 2025
desrosj added a commit that referenced this pull request Aug 19, 2025
This will be hanlded in #254 instead.
@desrosj desrosj requested a review from Crixu August 20, 2025 12:19
@desrosj desrosj merged commit 089bb58 into master Aug 26, 2025
@desrosj desrosj deleted the prevent-word-splitting-in-env-vars branch August 26, 2025 06:25
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.

3 participants