Skip to content

Fix multipart filename leak#478

Merged
ryanfowler merged 1 commit intomainfrom
fix-multipart-filename-leak
Apr 19, 2026
Merged

Fix multipart filename leak#478
ryanfowler merged 1 commit intomainfrom
fix-multipart-filename-leak

Conversation

@ryanfowler
Copy link
Copy Markdown
Owner

Summary

  • Send only filepath.Base(...) in multipart Content-Disposition filenames
  • Keep full paths for opening files and MIME detection
  • Add regression coverage for path-based uploads and update docs

Testing

  • go test -v ./internal/multipart
  • go test -v ./integration -run TestMain/multipart

@ryanfowler ryanfowler enabled auto-merge April 19, 2026 21:02
@ryanfowler ryanfowler merged commit 0c01c60 into main Apr 19, 2026
10 checks passed
@ryanfowler ryanfowler deleted the fix-multipart-filename-leak branch April 19, 2026 21:02
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