-
-
Notifications
You must be signed in to change notification settings - Fork 435
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Log a warning when rate limiting is active #3817
Comments
Let's double check if we log warning in non-debug mode too (and check what other SDKs are doing). |
Do you think it would also make sense to log a warning in case a single envelope item has been dropped due to rate-limiting? Similar to what has been suggested in here. Regarding the handling of additional noise, |
@rwachtler yes it'd make sense, thanks for pointing that out! |
Description
Whe should log this as an
warning
instead ofinfo
:sentry-java/sentry/src/main/java/io/sentry/transport/RateLimiter.java
Line 80 in 285450b
The text was updated successfully, but these errors were encountered: