Skip to content

Default AsyncContext timeout reduced to 30ms #282

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

Closed
aaron-steinfeld opened this issue Nov 30, 2020 · 0 comments · Fixed by #283
Closed

Default AsyncContext timeout reduced to 30ms #282

aaron-steinfeld opened this issue Nov 30, 2020 · 0 comments · Fixed by #283
Labels

Comments

@aaron-steinfeld
Copy link
Contributor

In #251 , the ability to configure the async request timeout was added, but it was defaulted to 30. Because that field is measured in ms, this changes the existing default behavior of a 30s timeout to a 30ms timeout - a significant breaking change for those using the async mode.

To Reproduce
Use the async servlet mode without configuring the timeout.

Expected behavior
Without setting a timeout the default behavior should remain unchanged at 30s.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant