-
Notifications
You must be signed in to change notification settings - Fork 107
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
Port content app aiohttp internals to use AppRunner #1876
Comments
From: pulpbot (pulpbot) PR: #610 |
From: @bmbouter (bmbouter) To test this I'm posting a 9000 char header to the content app which by default would only accept an 8190 byte payload.
As is this yields:
|
From: @RCMariko (rchan) Removing from current Sprint 80. Target for Sprint 82. Can be added back to Sprint 81 if staffing allows. |
From: @bmbouter (bmbouter) Moving back to NEW as I am not actively working on this. |
Related to #1911 |
This is not completed, but the only group who asked for it is Katello and they already have a patch resolving it for a long time now. I'm going to close for now, and if we need this, comment and we can reopen it. |
Author: @bmbouter (bmbouter)
Redmine Issue: 6397, https://pulp.plan.io/issues/6397
motivation
max_field_size
set so that the content app can receive larger certificates from clients.make_handler()
but that has been deprecated since 3.2 in favor ofApplication Runners
.Solution
Port the content app to use Application Runners
The text was updated successfully, but these errors were encountered: