Skip to content
This repository has been archived by the owner on Jul 13, 2023. It is now read-only.

NoMethodFound exception on invalid Content-Disposition header #2283

Closed
arokettu opened this issue Aug 23, 2016 · 1 comment · Fixed by #2284
Closed

NoMethodFound exception on invalid Content-Disposition header #2283

arokettu opened this issue Aug 23, 2016 · 1 comment · Fixed by #2284

Comments

@arokettu
Copy link
Contributor

arokettu commented Aug 23, 2016 •
edited
Loading

Tried to upload file by URL

The server returned
"content-disposition"=>"inline; filename=blabla.jpg"
filename must be quoted so the header is invalid
regexp match(/filename="([^"]*)"/) finds nothing here so match(...)[1] fails

I think that it's good to just ignore it instead of trying to parse it anyway
@arokettu arokettu mentioned this issue Aug 23, 2016
Merged
@arokettu
Copy link
Contributor Author

PR for fix by ignoring - #2284

tute pushed a commit that referenced this issue Aug 24, 2016
@arokettu @tute
Ignore invalid Content-Disposition header (#2284)
a49c59f 
* ignore invalid content-disposition header

[fixes #2283]
@jbielick jbielick mentioned this issue Aug 23, 2017
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Ignore invalid Content-Disposition header
1 participant
@arokettu