search

vy / log4j2-redis-appender

A Log4j 2 Appender for Redis.
Apache License 2.0
33 stars 10 forks source link

Bug: eventRateLimiter failure increments the wrong counter #50

Closed vliegveld5 closed 1 year ago

vliegveld5 commented 1 year ago

https://github.com/vy/log4j2-redis-appender/blob/3179c4995212c15815c0363fb98305860bf34d96/appender/src/main/java/com/vlkan/log4j2/redis/appender/RedisThrottler.java#L274

Should be:

       jmxBean.incrementEventRateLimitFailureCount(1);
vy commented 1 year ago

Thanks for the report @vliegveld5! Fixed in bede6c08fa78c0e771d87048338c4978cfb142eb.