-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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: size of email content "too large" #2680
fix: size of email content "too large" #2680
Conversation
NV-1580 Unable to send email if email content size is large
When email content is large, trigger fails with error One workaround is to use minified content, but it is not always a good solution. Expectation:- Size limit should be increased and exact value should be mentioned in docs. |
👇 Click on the image for a new way to code review
Legend |
app.use(extendedBodySizeRoutes, bodyParser.json({ limit: '20mb' })); | ||
app.use(extendedBodySizeRoutes, bodyParser.urlencoded({ limit: '20mb', extended: true })); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to double check, is this syntax allowed in nestjs? Not sure I have seen it elsewhere
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You mean the array of routes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It works @scopsy :)
What change does this PR introduce?
The 20mb was only applied to event trigger - not for the broadcast, test email or to save and update the email content.
Why was this change needed?
Other information (Screenshots)