logrus-cloudwatchlogs

logrus-cloudwatchlogs copied to clipboard

Interested in finding out how you found the magic number 26 to add to message size

3

Hi, Thanks for writing this package, it's really cool! I'm currently writing something similar for zerolog. When you work out the total size of the batch to send to AWS...

mec07

NewBatchingHook doesn't seem to grab the sequence token

4

I am attempting to use batching to avoid `CloudWatch` rate limiting. During my automated testing, I am seeing the following: ``` === RUN TestCloudWatchBatching100 Failed to fire hook: InvalidSequenceTokenException: The...

ramonkeypr

Unless I'm missing something, I think the batchFrequency is not honored when the log stream already exists : https://github.com/kdar/logrus-cloudwatchlogs/blob/master/hook.go#L46-L66 ``` // grab the next sequence token if len(resp.LogStreams) > 0...

jvelo