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

Sets client_max_body_size to 0, unlimited for nginx #99

Merged
merged 1 commit into from Jun 8, 2020

Conversation

dkliban
Copy link
Member

@dkliban dkliban commented Jun 5, 2020

The default value for client_max_body_size is 1mb. We need it to be unlimited.

Apache's equivalent is LimitRequestBody and it defaults to unlimited.

This patch updates the 'push' test to use a large image that would require this setting to be set.

This patch also adds an additional 'pull' operation to the 'push' test to verify that the pushed
image can be pulled back from the registry.

fixes: #6916
https://pulp.plan.io/issues/6916

@pulpbot
Copy link
Member

pulpbot commented Jun 5, 2020

Attached issue: https://pulp.plan.io/issues/6916

@dkliban dkliban force-pushed the client-max-body-size branch 2 times, most recently from 0a23017 to 1e1ce79 Compare June 5, 2020 20:46
The default value for client_max_body_size is 1mb. We need it to be unlimited.

Apache's equivalent is LimitRequestBody and it defaults to unlimited.

This patch updates the 'push' test to use a large image that would require this setting to be set.

This patch also adds an additional 'pull' operation to the 'push' test to verify that the pushed
image can be pulled back from the registry.

fixes: #6916
https://pulp.plan.io/issues/6916
@ipanova ipanova merged commit 1d07b61 into pulp:push-api Jun 8, 2020
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

3 participants