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.

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

update unreturnedConnectionTimeout description in documentation #160

Merged
merged 1 commit into from
Feb 16, 2024

Conversation

matteomazza91
Copy link

This is an opinable proposal.

I'd like to not see this option being discouraged as I think it's necessary for increased resiliency in prod-ready applications.
Still, the description should continue to point out that these kind of errors must be monitored and fixed.

@swaldman swaldman merged commit 6ac8be9 into swaldman:integration Feb 16, 2024
@swaldman
Copy link
Owner

Thanks!

See bce166e for my editorializing on your editorializing.

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

Successfully merging this pull request may close these issues.

2 participants