Skip to content

Fix curl command for text bodies starting with at sign#5155

Open
cyphercodes wants to merge 1 commit into
Orange-OpenSource:masterfrom
cyphercodes:cyphercodes/fix-5154-curl-data-raw
Open

Fix curl command for text bodies starting with at sign#5155
cyphercodes wants to merge 1 commit into
Orange-OpenSource:masterfrom
cyphercodes:cyphercodes/fix-5154-curl-data-raw

Conversation

@cyphercodes

Copy link
Copy Markdown

What

Fix the generated debug curl command for text request bodies that begin with @.

Why

curl --data $'@filename\n' treats the value as a filename and fails before sending the request. For text bodies starting with @, the generated command now uses --data-raw so curl sends the literal body.

Fixes #5154

Tests

  • cargo fmt --check
  • cargo test -p hurl http::curl_cmd --lib
  • ./target/debug/hurl --verbose build/repro-5154.hurl (local repro; verified generated command contains --data-raw $'@filename\\n' and not --data $'@filename\\n')

@lepapareil

Copy link
Copy Markdown
Collaborator

/rebase

@hurl-bot

Copy link
Copy Markdown
Collaborator

🕗 /rebase is running, please wait for completion.

@hurl-bot

Copy link
Copy Markdown
Collaborator

🔨 Auto rebase from Orange-OpenSource/hurl/master succeeds, cyphercodes/hurl/cyphercodes/fix-5154-curl-data-raw now embeds these commits:

  • 76caed6 Update crates
  • ee57fd3 Add http3 option in config_file
  • a2c72ca Fix Squid installation on Windows
  • cdc06ae Add http2 option in config_file
  • 1409b07 Add http1.0 option in config_file
  • 558c5f8 Add http1.1 option in config_file
  • fe7dd4d Add --variables-file per request option
  • cb54bb6 Add user option in config_file
  • f50728c Update CONTRIBUTING guide with AI tool use policy.
  • 3c3149b Add --no-jsonpath-coercion in [Options] section
  • 61c80bf Update to Rust 1.97.1

@hurl-bot
hurl-bot force-pushed the cyphercodes/fix-5154-curl-data-raw branch from c72226e to bdbe32c Compare July 24, 2026 15:28
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.

Hurl debug curl command is erroneous with text request body that start with @

3 participants