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

app.use(express.limit()) not working #1815

Closed
SamuelBolduc opened this issue Nov 13, 2013 · 6 comments
Closed

app.use(express.limit()) not working #1815

SamuelBolduc opened this issue Nov 13, 2013 · 6 comments

Comments

@SamuelBolduc
Copy link

I have written a StackOverflow reply which describes the problem in detail. It can be found here : http://stackoverflow.com/questions/19917401/node-js-express-request-entity-too-large

Is this a problem related to my implementation or really a framework bug? It seems it does not work for anyone, from what I saw while searching earlier about the problem...

@jonathanong
Copy link
Member

It's deprecated, don't use it.

You can set the limit in any of the connect parsers directly.

@SamuelBolduc
Copy link
Author

Actually I did try it too, if you read my post on S/O... Isn't app.use(express.bodyParser({limit: '50mb'})); what you're talking about?

@dougwilson
Copy link
Contributor

It is, but if you are using another bodyParser higher up (like a global one) it takes precedent since it is first and you're limit will be of that one.

@jonathanong
Copy link
Member

no, connect.json() and connect.urlencoded() no longer use the limit middleware. if you only used app.use(express.bodyParser({limit: '50mb'})) everything will work fine.

@sunil-ideas2it
Copy link

I am facing the issue request entity to large for that i am trying this fix but it is not working . Anyone knows how this fix this in MEAN IO framework. which configuration file i need to change in order to make it work .

@zionzinkerz
Copy link

sunil-ideas2it, read here:
http://stackoverflow.com/questions/29939852/mean-io-error-request-entity-too-large-how-to-increase-bodyparser-limit-ou

depending on the version of mean.io you are using, the settings might be in your /config/env/*.js files

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

5 participants