Skip to content

Quote ejson2env output in shell examples - #191

Merged
rafaelfranca merged 1 commit into
Shopify:mainfrom
plgarg:docs/quote-ejson2env-examples
Oct 8, 2026
Merged

rafaelfranca merged 1 commit into
Shopify:mainfrom
plgarg:docs/quote-ejson2env-examples

Conversation

@plgarg

@plgarg plgarg commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

What changed

Quoted the command substitution in the README and Ronn manpage examples:

eval "$(ejson2env ...)"

This prevents shell word splitting and pathname expansion from re-processing decrypted output before eval receives it.

Validation

  • go test ./...
  • Confirmed both documented examples are quoted
  • git diff --check

Assisted-By: devx/d19d2b77-3e88-4190-b891-719dc11d7bcc
@rafaelfranca
rafaelfranca merged commit e13a333 into Shopify:main Oct 8, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants