-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
fix-#358: Add the limit to the json payload #361
fix-#358: Add the limit to the json payload #361
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Umm... did someone forget to read the style guide? Fix that PR title and let's try again! @rushil-b-patel |
Hey @rushil-b-patel! Thanks for sticking to the guidelines! High five! 🙌🏻 |
@krishnaacharyaa ...! |
@rushil-b-patel you break the rules again :) |
…into fix/json-payload
should I have to wait until the issue is being confirmed and then PR...! |
Summary
Added the limit to the json payload that is being sent by the user.
Description
The actual payload that user can submit to your app /api should be restricted to avoid unnecessary behaviour/ overload of the data. Therefore there should be a limit to the body payload.
app.use(express.json( { limit: '50kb' })); //we can change set the max limit according to the usage.
Issue(s) Addressed
Prerequisites