Do not jump over semicolons when filename contains a path #1964

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Contributor

cuklev commented Oct 7, 2017

Content-Disposition: attachment; filename=correct.jpg; somethingelse=/wrong.jpg
curl -O -J saves to wrong.jpg when it should probably save to correct.jpg.

@cuklev cuklev changed the title from Do not jump over seimicolons when filename contains a path to Do not jump over semicolons when filename contains a path Oct 7, 2017

bagder approved these changes Oct 9, 2017

Looks good. I also updated test 1422 to verify this and will amend your commit with it when I merge!

Owner

bagder commented Oct 9, 2017

Thanks!

@bagder bagder closed this in c95c92d Oct 9, 2017

@cuklev cuklev deleted the cuklev:content-disposition-filename-parse branch Dec 5, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment