Skip to content

Return 400 for invalid UTF-8 multipart fields#1239

Merged
lovasoa merged 1 commit intomainfrom
bad-request-utf8
Mar 12, 2026
Merged

Return 400 for invalid UTF-8 multipart fields#1239
lovasoa merged 1 commit intomainfrom
bad-request-utf8

Conversation

@lovasoa
Copy link
Collaborator

@lovasoa lovasoa commented Mar 12, 2026

Summary

  • return 400 Bad Request when a multipart text field contains invalid UTF-8
  • add a regression test covering the curl repro for malformed multipart payloads
  • document the fix in CHANGELOG.md

Verification

  • cargo fmt --all
  • cargo clippy --all-targets --all-features -- -D warnings
  • cargo test test_invalid_utf8_multipart_text_field_returns_bad_request -- --nocapture
  • cargo test test_file_upload_direct -- --nocapture

Repro

curl --http1.1 -X POST http://127.0.0.1:6281/tests/requests/variables.sql -H 'Content-Type: multipart/form-data; boundary=xxx' --data-binary @/tmp/sqlpage-invalid-multipart.bin

@lovasoa lovasoa marked this pull request as ready for review March 12, 2026 10:51
@lovasoa lovasoa merged commit c7e5be7 into main Mar 12, 2026
13 checks passed
@lovasoa lovasoa deleted the bad-request-utf8 branch March 12, 2026 10:57
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