Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Don't close connection for in-process request #41

Closed
bpbecker opened this issue Mar 17, 2023 · 0 comments · Fixed by #43
Closed

Don't close connection for in-process request #41

bpbecker opened this issue Mar 17, 2023 · 0 comments · Fixed by #43
Assignees
@bpbecker

Comments

@bpbecker
Copy link
Contributor

The problem:
Jarvis has a "ConnectionTimeout" setting intended to clean up connections that have gone dormant. The default value is 30 seconds. Currently, we may inadvertently close a connect for a request to endpoint that runs for longer than ConnectionTimeout.

The suggestion:
Add a "in-progress" flag to the connection namespace that the connection reaper will respect and not close connections for requests that are being serviced.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bpbecker bpbecker

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix Don't close connection for in-process request #41 Dyalog/Jarvis
1 participant
@bpbecker