Percent-encoding in filenames not handled correctly #266

Closed
opened 2026-01-19 18:29:40 +00:00 by michael · 2 comments
Owner

Originally created by @JustAnotherArchivist on GitHub.

Percent-encoding in filenames is not handled correctly:

> echo foo > test%C3%A4test
> curl --upload-file test%C3%A4test https://{redacted}/;echo
https://{redacted}/LN3I1/test%C3%A4test

Loading that URL produces a 404 because the file was actually uploaded as https://{redacted}/LN3l1/test%25C3%25A4test instead, i.e. with the per cent symbols encoded as %25. That's probably the correct way to handle such uploads, but the output of the PUT request should also reflect it.

Originally created by @JustAnotherArchivist on GitHub. Percent-encoding in filenames is not handled correctly: > echo foo > test%C3%A4test > curl --upload-file test%C3%A4test https://{redacted}/;echo https://{redacted}/LN3I1/test%C3%A4test Loading that URL produces a 404 because the file was actually uploaded as `https://{redacted}/LN3l1/test%25C3%25A4test` instead, i.e. with the per cent symbols encoded as `%25`. That's probably the correct way to handle such uploads, but the output of the PUT request should also reflect it.
Author
Owner

@paolafrancesca commented on GitHub:

you're right @JustAnotherArchivist : the current solution was a little clumsy. I fixed with a more straightforward one in #216 , can you give a try?

@paolafrancesca commented on GitHub: you're right @JustAnotherArchivist : the current solution was a little clumsy. I fixed with a more straightforward one in #216 , can you give a try?
Author
Owner

@JustAnotherArchivist commented on GitHub:

Thanks, I can confirm that #216 fixes this.

@JustAnotherArchivist commented on GitHub: Thanks, I can confirm that #216 fixes this.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: dutchcoders/transfer.sh#266