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

Increase Body Parser Limit to Fix 413 Error on update-template Endpoint #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

BertramRay
Copy link

While invoking the update-template API, I encountered a 413 Payload Too Large HTTP error code. Upon investigation, it appears that the size of the form-data in the request was exceeding server limits, despite being only approximately 70kb in size.

This issue is particularly relevant when using no-code email editors, as the emails generated can easily reach or exceed this threshold. Reviewing the config/bodyParser.js configuration file, I noticed that there was no explicit limit set for the body size, which led me to believe that the default limit imposed by the framework was too low for our use case.

To address this problem, I have increased the body parser limit to 10mb. This change effectively resolves the issue and allows emails of larger sizes to be processed without encountering a 413 error.

Please consider merging this adjustment to ensure smooth operation of the update-template endpoint for all users.

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

Successfully merging this pull request may close these issues.

None yet

1 participant