fluent / fluent-plugin-s3

Amazon S3 input and output plugin for Fluentd
https://docs.fluentd.org/output/s3
314 stars 218 forks source link

Revert wrong fix in #388 #396

Closed ashie closed 2 years ago

ashie commented 2 years ago

Aws::AssumeRoleCredentials.new and Aws::AssumeRoleWebIdentityCredentials.new expect a hash argument, not kwargs. Tests for them were wrong, not the implementation.

https://docs.aws.amazon.com/sdk-for-ruby/v3/api/Aws/AssumeRoleCredentials.html#initialize-instance_method https://docs.aws.amazon.com/sdk-for-ruby/v3/api/Aws/AssumeRoleWebIdentityCredentials.html#initialize-instance_metho

Signed-off-by: Takuro Ashie ashie@clear-code.com

ashie commented 2 years ago

Sorry for that...