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

permit bodyParserLimitSize configuration by ENV variable #4985

Merged
merged 3 commits into from
Feb 12, 2022

Conversation

matthgyver
Copy link
Contributor

set bodyParserLimitSize by default to 1mb
permit modifications by environment variable

set bodyParserLimitSize by default to 1mb
permit modifications by environment variable
@auto-assign auto-assign bot requested a review from NGPixel February 8, 2022 08:50
@matthgyver matthgyver changed the title permit bodyParserLimitSize configuration permit bodyParserLimitSize configuration by ENV variable Feb 8, 2022
@NGPixel NGPixel merged commit 7988fa6 into requarks:main Feb 12, 2022
@NGPixel
Copy link
Member

NGPixel commented Feb 12, 2022

I modified your PR to use a config value instead of ENV directly, so that it can be used from both the config.yml or ENV.

jionggyu pushed a commit to jionggyu/wiki-2.5.302-patch that referenced this pull request Jul 9, 2024
* permit bodyParserLimitSize configuration

set bodyParserLimitSize by default to 1mb
permit modifications by environment variable

* fix: use bodyparserlimit config with fallback default

Co-authored-by: Nicolas Giard <[email protected]>
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.

2 participants