krishnaacharyaa / wanderlust

WanderLust is a MERN travel blog website 🚀 This project is aimed to help people to contribute in open source, upskill in react and master git.
https://wanderlust-beta.vercel.app/
MIT License
221 stars 1.25k forks source link

fix-#358: Add the limit to the json payload #361

Closed rushil-b-patel closed 6 months ago

rushil-b-patel commented 6 months ago

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

vercel[bot] commented 6 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
wanderlust ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 26, 2024 7:21am
wanderlust-backend ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 26, 2024 7:21am
github-actions[bot] commented 6 months ago

Umm... did someone forget to read the style guide? Fix that PR title and let's try again! @rushil-b-patel Don't worry, it happens to the best of us! Check out our contributor guidelines for more details.

github-actions[bot] commented 6 months ago

Hey @rushil-b-patel! Thanks for sticking to the guidelines! High five! 🙌🏻

rushil-b-patel commented 6 months ago

@krishnaacharyaa ...!

krishnaacharyaa commented 6 months ago

@rushil-b-patel you break the rules again :) It was clearly said to get the issue assigned and not to surprise with any PRs :)

rushil-b-patel commented 6 months ago

should I have to wait until the issue is being confirmed and then PR...! @krishnaacharyaa