Make sequence token optional

fluent-plugins-nursery / fluent-plugin-cloudwatch-logs

CloudWatch Logs Plugin for Fluentd

MIT License

201 stars 141 forks source link

Make sequence token optional #250

Open gdziwoki opened 1 year ago

gdziwoki commented 1 year ago

Problem

https://aws.amazon.com/about-aws/whats-new/2023/01/amazon-cloudwatch-logs-log-stream-transaction-quota-sequencetoken-requirement/

Sequence tokens are no longer required. Would be nice to have a switch to make them optional and get rid of those pesky InvalidSequenceTokenException.

petergvizd commented 1 year ago

Looks like InvalidSequenceTokenException should not appear anymore according to https://docs.aws.amazon.com/AmazonCloudWatchLogs/latest/APIReference/API_PutLogEvents.html . But anyway there is prepared PR for getting rid of it in code https://github.com/fluent-plugins-nursery/fluent-plugin-cloudwatch-logs/pull/251