-
Notifications
You must be signed in to change notification settings - Fork 52
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
Use fedora 33, set large --max-http-header-size #39
base: master
Are you sure you want to change the base?
Conversation
Fedora 29 has nodejs 10.16.3 which is EOL. Fedora 33 (and 34) currently has 14.16.1. The newer NodeJS improves RedHatInsights#38 by returning 431 Request Header Fields Too Large instead of 400 Bad Request that NodeJS 10 returned.
64KB is larger than most server limits, but as a proxy, it's not insights-proxy's job to set limits, let backend do it.
New image size is 403MB, down from 616MB. |
Nice! LGTM |
Tested locally, I'm getting
|
Looks like this is a problem in the Akamai configuration. The response for the ESI snippets contains both Newer NodeJS refuses to parse this response as having both these headers together could allow for an HTTP request smuggling attack. @karelhala is there any way this can be fixed in the Akamai side? otherwise we'll have to keep insights-proxy using an old version of NodeJS forever curl output (click to expand)
|
Q: why is it fetching the snippet back from the proxy — https://prod.foo.redhat.com:1337/apps/chrome/snippets/head.html ?
|
All the requests are going trough the proxy even though they are then fetched from cloud.redhat.com. I'll take a look what could be done with it. |
Closes #38.