Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

File upload cookbook limited to small filesizes #12

Open
lightraven24 opened this issue Sep 17, 2016 · 6 comments
Open

File upload cookbook limited to small filesizes #12

lightraven24 opened this issue Sep 17, 2016 · 6 comments

Comments

@lightraven24
Copy link
Contributor

The file upload cookbook should set maximumContentLength as it cuts out at the default 2MB right now.

Also, it doesn't gracefully handle the 413 - it just fails with "connection reset".

@psibi
Copy link
Member

psibi commented Sep 17, 2016

Can you send a PR fixing that ?

@lightraven24
Copy link
Contributor Author

Created #14

Though, I'm not sure how to solve the 413 issue.

@psibi
Copy link
Member

psibi commented Sep 17, 2016

Thanks for the PR! I tried with the resolver 6.11 and I was getting 413 status code as expected.

127.0.0.1 - - [17/Sep/2016:20:16:53 +0530] "POST / HTTP/1.1" 413 - "http://localhost:3000/" "Mozilla/5.0 (X11; Linux x86_64; rv:48.0) Gecko/20100101 Firefox/48.0"

I guess you are probably using an old version of the package ?

@lightraven24
Copy link
Contributor Author

lightraven24 commented Sep 17, 2016

I also get a 413 - but do you get an error page, or do you just get a "connection reset" in the browser?

I'm using the stock packages in Ubuntu 16.04, don't seem to have a resolver package - how can I check that?

@psibi
Copy link
Member

psibi commented Sep 17, 2016

Okay, so I'm getting the connection reset in Firefox. But for Google Chrome it properly shows the error message.

I'm guessing this maybe an issue with firefox: https://bugzilla.mozilla.org/show_bug.cgi?id=746877

@psibi
Copy link
Member

psibi commented Sep 27, 2016

@lightraven24 Can you confirm if this is working for you in Chrome ? Probably this should be filed as a bug in the Yesod repository.

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

No branches or pull requests

2 participants