-
Notifications
You must be signed in to change notification settings - Fork 531
fix(docker): increase jetty request header size #12048
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
Conversation
Increase request header size from 8192 to 10240 to match manual Solr installation instructions. Prevents errors when importing larger metadata blocks.
|
I added this to the agenda to discuss on Thursday: https://docs.google.com/document/d/1zYk_bLV9qB8YKQmJXBcsDz_OrSLY3Cx8cPGxJ3oFRQY/edit?usp=sharing |
|
I have a few other useful bits that come in handy. We could add them: Unfortunately, using SOLR_JETTY_REQUEST_HEADER_SIZE=102400 doesn't seem to work (may be a bug, as the docs indicate otherwise). |
|
@Athemis thanks for the PR! Merged! As discussed during the containerization meeting today, we'll think about adding more flags in the future. |
What this PR does / why we need it:
This PR sets an environment variable for the Solr docker container, increasing jetty response header size from the default value of
8192to102400as recommended in the installation guide. This prevents errors when importing larger metadata blocks.Which issue(s) this PR closes:
Special notes for your reviewer:
Suggestions on how to test this:
developbranch or6.8Does this PR introduce a user interface change? If mockups are available, please link/include them here:
no
Is there a release notes update needed for this change?:
Additional documentation: