Change rate limiting logs to level warning or error

getsentry / sentry-cocoa

The official Sentry SDK for iOS, tvOS, macOS, watchOS.

https://sentry.io/for/cocoa/

MIT License

804 stars 317 forks source link

Change rate limiting logs to level warning or error #4456

Open philipphofmann opened 3 days ago

philipphofmann commented 3 days ago

Description

See https://github.com/getsentry/team-mobile/issues/195.

rwachtler commented 6 hours ago

Hey @philipphofmann ! I was wondering if the goal is just to update the log-level here to something like SENTRY_LOG_WARN(@"RateLimit is active for all envelope items.") (as the title of this issue states) or also have some additional log statement, which isn't bound to the condition that all items has to be rate-limited in order to print the log?

I've the first variant ready in a fork, let me know if I should open a PR.

`SentryHttpTransportTests`