fix: Errors shortly after SentrySDK.init don't affect the session

CHANGELOG.md

@@ -8,6 +8,10 @@ This version adds a dependency on Swift.
 
 - Properly demangle Swift class name (#2162)
 
+### Fixes
+
+- Errors shortly after SentrySDK.init now affect the session (#2430)
+
 ### Breaking Changes
 
 - Remove `- [SentryOptions initWithDict:didFailWithError:]` (#2404)

Sources/Sentry/SentryHub.m

@@ -15,6 +15,7 @@
 #import "SentrySamplingContext.h"
 #import "SentryScope.h"
 #import "SentrySerialization.h"
+#import "SentrySession+Private.h"
 #import "SentryTracer.h"
 #import "SentryTracesSampler.h"
 #import "SentryTransaction.h"
@@ -33,6 +34,7 @@
 @property (nonatomic, strong) id<SentryCurrentDateProvider> currentDateProvider;
 @property (nonatomic, strong) NSMutableArray<id<SentryIntegrationProtocol>> *installedIntegrations;
 @property (nonatomic, strong) NSMutableSet<NSString *> *installedIntegrationNames;
+@property (nonatomic) NSUInteger errorsBeforeSession;
 
 @end
 
@@ -53,6 +55,7 @@ - (instancetype)initWithClient:(nullable SentryClient *)client
         _installedIntegrationNames = [[NSMutableSet alloc] init];
         _crashWrapper = [SentryCrashWrapper sharedInstance];
         _tracesSampler = [[SentryTracesSampler alloc] initWithOptions:client.options];
+        _errorsBeforeSession = 0;
 #if SENTRY_TARGET_PROFILING_SUPPORTED
         if (client.options.isProfilingEnabled) {
             _profilesSampler = [[SentryProfilesSampler alloc] initWithOptions:client.options];
@@ -93,6 +96,11 @@ - (void)startSession
         }
         _session = [[SentrySession alloc] initWithReleaseName:options.releaseName];
 
+        if (_errorsBeforeSession > 0) {
+            _session.errors = _errorsBeforeSession;
+            _errorsBeforeSession = 0;
+        }
+
         NSString *environment = options.environment;
         if (nil != environment) {
             _session.environment = environment;
@@ -447,6 +455,7 @@ - (SentryId *)captureError:(NSError *)error withScope:(SentryScope *)scope
                               withScope:scope
                  incrementSessionErrors:^(void) { return [self incrementSessionErrors]; }];
         } else {
+            _errorsBeforeSession++;
             return [client captureError:error withScope:scope];
         }
     }
@@ -468,6 +477,7 @@ - (SentryId *)captureException:(NSException *)exception withScope:(SentryScope *
                                   withScope:scope
                      incrementSessionErrors:^(void) { return [self incrementSessionErrors]; }];
         } else {
+            _errorsBeforeSession++;
             return [client captureException:exception withScope:scope];
         }
     }

Sources/Sentry/SentrySession.m

@@ -22,6 +22,11 @@
     }
 }
 
+@interface
+SentrySession ()
+@property (nonatomic) NSUInteger errors;
+@end
+
 @implementation SentrySession
 
 @synthesize flagInit = _init;

Sources/Sentry/include/SentrySession+Private.h

@@ -8,6 +8,8 @@ NSString *nameForSentrySessionStatus(SentrySessionStatus status);
 @interface
 SentrySession (Private)
 
+@property (nonatomic) NSUInteger errors;
+
 - (void)setFlagInit;
 
 @end

Tests/SentryTests/SentryHubTests.swift

@@ -394,6 +394,19 @@ class SentryHubTests: XCTestCase {
         XCTAssertEqual(1, fixture.client.captureSessionInvocations.count)
     }
 
+    func testCaptureErrorBeforeSession() {
+        let sut = fixture.getSut()
+        sut.capture(error: fixture.error, scope: fixture.scope).assertIsNotEmpty()
+        sut.startSession()
+
+        XCTAssertEqual(fixture.client.captureErrorWithScopeInvocations.count, 1)
+        XCTAssertEqual(fixture.client.captureSessionInvocations.count, 1)
+
+        if let session = fixture.client.captureSessionInvocations.first {
+            XCTAssertEqual(session.errors, 1)
+        }
+    }
+
     func testCaptureWithoutIncreasingErrorCount() {
         let sut = fixture.getSut()
         sut.startSession()