Print a warning if the rate limit was reached (#2595)

CHANGELOG.md

@@ -7,6 +7,10 @@
 - Add SentryReplayQuality setting (`options.experimental.replay.quality`) ([#2582](https://github.com/getsentry/sentry-dart/pull/2582))
 - SPM Support ([#2280](https://github.com/getsentry/sentry-dart/pull/2280))
 
+### Enhancements
+
+- Print a warning if the rate limit was reached ([#2595](https://github.com/getsentry/sentry-dart/pull/2595))
+
 ### Fixes
 
 - WASM compat for Drift ([#2580](https://github.com/getsentry/sentry-dart/pull/2580))

dart/lib/src/transport/http_transport.dart

@@ -50,6 +50,10 @@ class HttpTransport implements Transport {
     if (response.statusCode == 200) {
       return _parseEventId(response);
     }
+    if (response.statusCode == 429) {
+      _options.logger(
+          SentryLevel.warning, 'Rate limit reached, failed to send envelope');
+    }
     return SentryId.empty();
   }
 

dart/test/transport/http_transport_test.dart

@@ -83,6 +83,10 @@ void main() {
       expect(mockRateLimiter.errorCode, 429);
       expect(mockRateLimiter.retryAfterHeader, '1');
       expect(mockRateLimiter.sentryRateLimitHeader, isNull);
+
+      expect(fixture.loggedLevel, SentryLevel.warning);
+      expect(
+          fixture.loggedMessage, 'Rate limit reached, failed to send envelope');
     });
 
     test('sentryRateLimitHeader', () async {
@@ -207,6 +211,10 @@ void main() {
       await sut.send(envelope);
 
       expect(fixture.clientReportRecorder.discardedEvents.isEmpty, isTrue);
+
+      expect(fixture.loggedLevel, SentryLevel.warning);
+      expect(
+          fixture.loggedMessage, 'Rate limit reached, failed to send envelope');
     });
 
     test('does record lost event for error >= 500', () async {
@@ -237,6 +245,8 @@ class Fixture {
   late var clientReportRecorder = MockClientReportRecorder();
 
   HttpTransport getSut(http.Client client, RateLimiter rateLimiter) {
+    options.debug = true;
+    options.logger = mockLogger;
     options.httpClient = client;
     options.recorder = clientReportRecorder;
     options.clock = () {
@@ -254,4 +264,18 @@ class Fixture {
     final tracer = SentryTracer(context, MockHub());
     return SentryTransaction(tracer);
   }
+
+  SentryLevel? loggedLevel;
+  String? loggedMessage;
+
+  void mockLogger(
+    SentryLevel level,
+    String message, {
+    String? logger,
+    Object? exception,
+    StackTrace? stackTrace,
+  }) {
+    loggedLevel = level;
+    loggedMessage = message;
+  }
 }