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
Increase proxy limit from 30s to 60s #39814
Conversation
Ideally this value should be configurable to match any reverse proxy configuration. For now increasing the limit to reduce the amount of errors in people's deployments. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, the 30s limit here is intentional to match edge defaults like Vercel, we can add an experimental config like experimental.proxyTimeout
to allow configuring this though.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, I'm gonna close this for now per-above, feel free to open a fresh PR with the mentioned experimental config though!
@abriginets are you planning to raise a PR for this? If not then I will raise one 😁 |
I'm currently busy with some other work so it would be great if you can open a PR. |
Thanks for the quick confirmation on this. I will raise a PR later tonight 👍 |
Bug
fixes #number
contributing.md
Fixes #36251
Increasing the limit since original issue wasn't addressed in 4 months.
60_000
value was picked based on default nginx values. See proxy_connect_timeout, proxy_read_timeout and proxy_send_timeout.