Skip to content

Conversation

@Watson1978
Copy link
Contributor

Fix #130

Currently, when using the default value for refresh_credentials_interval,
the string "5h" is passed to the interval argument of timer_execute method.

However, interval argument supports integer or float value.

Ref. https://docs.fluentd.org/plugin-helper-overview/api-plugin-helper-timer#timer_execute-title-interval-repeat-true-and-block

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
@Watson1978 Watson1978 requested review from ashie, daipom and kenhys June 27, 2025 08:16
@daipom
Copy link
Contributor

daipom commented Jun 27, 2025

Thanks!

The type of refresh_credentials_interval is time, but the default value should be the raw value.

@daipom daipom merged commit aa793d9 into fluent:main Jun 27, 2025
15 checks passed
@Watson1978 Watson1978 deleted the refresh_credentials_interval branch June 27, 2025 08:47
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.

AWS credentials are refreshed every 5 seconds instead of 5 hours, which is default value ofrefresh_credentials_interval

2 participants