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

nginx client_body_buffer_size #350

Open
justinrush opened this issue May 9, 2022 · 0 comments
Open

nginx client_body_buffer_size #350

justinrush opened this issue May 9, 2022 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed keepalive Keeps Issue's from beeing closed

Comments

@justinrush
Copy link

Since you expose nginx.config.client_max_body_size as a top level config it is probably best to also expose the client_body_buffer_size so folks can control what ends up being disk buffered and what doesn't. In addition to the performance benefit, setting that value high enough will also help prevent things like falco from endlessly complaining about unauthorized disk accesses.

In our environment I add the directive to: .nginx.config.httpSnippet. That works fine, but at first I had this as a top level config setting since I saw the other one in there.

@nschad nschad added help wanted Extra attention is needed good first issue Good for newcomers enhancement New feature or request labels May 10, 2022
@nschad nschad added the keepalive Keeps Issue's from beeing closed label Jun 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed keepalive Keeps Issue's from beeing closed
Projects
None yet
Development

No branches or pull requests

2 participants