Skip to content

[codex] Escape FakeRequest URLs and curl commands - #40

Merged
k-kinzal merged 1 commit into
mainfrom
codex/escape-fake-request-url-and-curl
Jul 4, 2026
Merged

[codex] Escape FakeRequest URLs and curl commands#40
k-kinzal merged 1 commit into
mainfrom
codex/escape-fake-request-url-and-curl

Conversation

@k-kinzal

@k-kinzal k-kinzal commented Jul 4, 2026

Copy link
Copy Markdown
Owner

Summary

  • URL-encode FakeRequest path parameter values as path segments so spaces, slashes, and apostrophes do not corrupt the generated URL.
  • Keep query parameters encoded through Guzzle's query builder while preserving raw values in toPsr7() query params.
  • Shell-quote generated curl URL, header, and body arguments so single quotes in user data produce valid commands.

Validation

  • composer test
  • composer lint

@k-kinzal
k-kinzal marked this pull request as ready for review July 4, 2026 16:23
@k-kinzal
k-kinzal merged commit 150b00a into main Jul 4, 2026
57 checks passed
@k-kinzal
k-kinzal deleted the codex/escape-fake-request-url-and-curl branch July 4, 2026 16:23
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