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

fix: fix fluentd in_http plugin keepalive_timeout option #968

Merged
merged 1 commit into from Oct 30, 2023

Conversation

cw-Guo
Copy link
Contributor

@cw-Guo cw-Guo commented Oct 29, 2023

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #965

Does this PR introduced a user-facing change?

None

Additional documentation, usage docs, etc.:


Signed-off-by: juicer <cwguoz@gmail.com>
@cw-Guo
Copy link
Contributor Author

cw-Guo commented Oct 29, 2023

Leave the variable KeepLiveTimeout as it is to avoid user change. It may be better to rename it to KeepaliveTimeout

@cw-Guo cw-Guo changed the title fix fd in_http plugin keepalive_timeout option fix: fix fluentd in_http plugin keepalive_timeout option Oct 29, 2023
@benjaminhuo benjaminhuo merged commit aa87188 into fluent:master Oct 30, 2023
6 checks passed
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.

bug: http input keepaliveTimeout in CRD transforms to keepLive_timeout, should be keepalive_timeout
2 participants