-
Notifications
You must be signed in to change notification settings - Fork 87
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
feat(config): Add a configuration for the keepalive_timeout #1645
Conversation
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.
Thank you for this addition. I'm happy to accept this PR, please see the comments attached.
@jan-auer The code has been modified, can you take a look at it? |
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.
Please add a changelog entry.
@iker-barriocanal A changelog has been added |
Co-authored-by: Jan Michael Auer <account@jauer.org>
@jan-auer Did a ci mission fail? |
Head branch was pushed to by a user without write access
CHANGELOG.md
Outdated
@@ -15,6 +15,7 @@ | |||
- Add OpenTelemetry Context. ([#1617](https://github.com/getsentry/relay/pull/1617)) | |||
- Add `app.in_foreground` and `thread.main` flag to protocol. ([#1578](https://github.com/getsentry/relay/pull/1578)) | |||
- Add support for View Hierarchy attachment_type. ([#1642](https://github.com/getsentry/relay/pull/1642)) | |||
- Add support for `keepalive_timeout` configuration. ([#1645](https://github.com/getsentry/relay/pull/1645)) |
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.
Could you, please, move this CHANGELOG entry to Features
in Unreleased
section?
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.
done
Open the keepalive_timeout configuration of relay server, when the IdleConnTimeout of the client is smaller than the keepalive_timeout of the server, the server will actively disconnect, then the client will report some unnecessary errors, expect to open the keepalive_timeout configuration to solve these problems