Merge pull request #318 from bugsnag/next

Next release

Author: fractalwrench

CHANGELOG.md

@@ -1,6 +1,41 @@
 Changelog
 =========
 
+## 2.15.0 (2019-03-07)
+
+* Add stopSession() and resumeSession() to Client [#314](https://github.com/bugsnag/bugsnag-react-native/pull/314)
+
+* (Android) Upgrade to bugsnag-android v4.12.0
+
+  ### Enhancements
+
+  * Add stopSession() and resumeSession() to Client
+  [#429](https://github.com/bugsnag/bugsnag-android/pull/429)
+
+  ### Bug fixes
+
+  * Prevent overwriting config.projectPackages if already set
+    [#428](https://github.com/bugsnag/bugsnag-android/pull/428)
+
+  * Fix incorrect session handledCount when notifying in quick succession
+    [#434](https://github.com/bugsnag/bugsnag-android/pull/434)
+
+* (Cocoa) Upgrade to bugsnag-cocoa v5.19.0
+
+  Note for Carthage users: this release updates the Xcode configuration to the settings recommended by Xcode 10.
+
+  * Update workspace to recommended settings suggested by XCode 10
+    [#324](https://github.com/bugsnag/bugsnag-cocoa/pull/324)
+
+  ### Enhancements
+
+  * Add stopSession() and resumeSession() to Bugsnag
+    [#325](https://github.com/bugsnag/bugsnag-cocoa/pull/325)
+
+  * Capture basic report diagnostics in the file path in case of crash report
+    content corruption
+    [#327](https://github.com/bugsnag/bugsnag-cocoa/pull/327)
+
 ## 2.14.0 (2019-01-23)
 
 * (Android) Upgrade to bugsnag-android v4.11.0

README.md

@@ -1,5 +1,5 @@
 # Bugsnag exception reporter for React Native
-[![Documentation](https://img.shields.io/badge/documentation-2.14.0-blue.svg)](http://docs.bugsnag.com/platforms/react-native/)
+[![Documentation](https://img.shields.io/badge/documentation-2.15.0-blue.svg)](http://docs.bugsnag.com/platforms/react-native/)
 
 Automatic [React Native crash reporting](https://www.bugsnag.com/platforms/react-native-error-reporting/) with Bugsnag helps you detect both native OS and JavaScript errors in your React Native apps.
 

android/build.gradle

@@ -18,12 +18,12 @@ android {
         minSdkVersion safeExtGet('minSdkVersion', 16)
         targetSdkVersion safeExtGet('targetSdkVersion', 26)
         versionCode 4
-        versionName '2.14.0'
+        versionName '2.15.0'
         consumerProguardFiles 'proguard-rules.pro'
     }
 }
 
 dependencies {
-    compile 'com.bugsnag:bugsnag-android:4.11.0'
+    compile 'com.bugsnag:bugsnag-android:4.12.0'
     compile 'com.facebook.react:react-native:+'
 }

android/src/main/java/com/bugsnag/BugsnagReactNative.java

@@ -84,6 +84,16 @@ public void startSession() {
       Bugsnag.startSession();
   }
 
+  @ReactMethod
+  public void stopSession() {
+      Bugsnag.stopSession();
+  }
+
+  @ReactMethod
+  public void resumeSession() {
+      Bugsnag.resumeSession();
+  }
+
   @ReactMethod
   public void startWithOptions(ReadableMap options) {
       String apiKey = null;
@@ -278,7 +288,7 @@ private void configureRuntimeOptions(Client client, ReadableMap options) {
               // The launch event session is skipped because autoCaptureSessions
               // was not set when Bugsnag was first initialized. Manually sending a
               // session to compensate.
-              client.startSession();
+              client.resumeSession();
           }
       }
   }

cocoa/BugsnagReactNative.h

@@ -38,5 +38,7 @@
 - (void)setUser:(NSDictionary *)userInfo;
 - (void)clearUser;
 - (void)startSession;
+- (void)stopSession;
+- (void)resumeSession;
 
 @end

cocoa/BugsnagReactNative.m

@@ -209,6 +209,20 @@ + (void)startWithConfiguration:(BugsnagConfiguration *)config {
     [Bugsnag startSession];
 }
 
+RCT_EXPORT_METHOD(stopSession) {
+    if (![Bugsnag bugsnagStarted]) {
+        return;
+    }
+    [Bugsnag stopSession];
+}
+
+RCT_EXPORT_METHOD(resumeSession) {
+    if (![Bugsnag bugsnagStarted]) {
+        return;
+    }
+    [Bugsnag resumeSession];
+}
+
 RCT_EXPORT_METHOD(clearUser) {
     if (![Bugsnag bugsnagStarted]) {
         return;
@@ -280,7 +294,7 @@ + (void)startWithConfiguration:(BugsnagConfiguration *)config {
         // The launch event session is skipped because shouldAutoCaptureSessions
         // was not set when Bugsnag was first initialized. Manually sending a
         // session to compensate.
-        [Bugsnag startSession];
+        [Bugsnag resumeSession];
     }
 }
 

cocoa/vendor/bugsnag-cocoa/Source/Bugsnag.h

@@ -211,13 +211,60 @@ static NSString *_Nonnull const BugsnagSeverityInfo = @"info";
     (BOOL)writeBinaryImagesForUserReported;
 
 /**
- * Manually starts tracking a new session.
+ * Starts tracking a new session.
  *
- * Sessions automatically start when the application enters the foreground state, and end when the application exits
- * the foreground.If you wish to manually start sessions, simply call this method from the relevant part of your
- * application. Starting a new session will automatically end the previous one.
+ * By default, sessions are automatically started when the application enters the foreground.
+ * If you wish to manually call startSession at
+ * the appropriate time in your application instead, the default behaviour can be disabled via
+ * shouldAutoCaptureSessions.
+ *
+ * Any errors which occur in an active session count towards your application's
+ * stability score. You can prevent errors from counting towards your stability
+ * score by calling stopSession and resumeSession at the appropriate
+ * time in your application.
+ *
+ * @see stopSession:
+ * @see resumeSession:
  */
-
 + (void)startSession;
 
+/**
+ * Stops tracking a session.
+ *
+ * When a session is stopped, errors will not count towards your application's
+ * stability score. This can be advantageous if you do not wish these calculations to
+ * include a certain type of error, for example, a crash in a background service.
+ * You should disable automatic session tracking via shouldAutoCaptureSessions if you call this method.
+ *
+ * A stopped session can be resumed by calling resumeSession,
+ * which will make any subsequent errors count towards your application's
+ * stability score. Alternatively, an entirely new session can be created by calling startSession.
+ *
+ * @see startSession:
+ * @see resumeSession:
+ */
++ (void)stopSession;
+
+/**
+ * Resumes a session which has previously been stopped, or starts a new session if none exists.
+ *
+ * If a session has already been resumed or started and has not been stopped, calling this
+ * method will have no effect. You should disable automatic session tracking via
+ * shouldAutoCaptureSessions if you call this method.
+ *
+ * It's important to note that sessions are stored in memory for the lifetime of the
+ * application process and are not persisted on disk. Therefore calling this method on app
+ * startup would start a new session, rather than continuing any previous session.
+ *
+ * You should call this at the appropriate time in your application when you wish to
+ * resume a previously started session. Any subsequent errors which occur in your application
+ * will be reported to Bugsnag and will count towards your application's stability score.
+ *
+ * @see startSession:
+ * @see stopSession:
+ *
+ * @return true if a previous session was resumed, false if a new session was started.
+ */
++ (BOOL)resumeSession;
+
 @end

cocoa/vendor/bugsnag-cocoa/Source/Bugsnag.m

@@ -226,6 +226,20 @@ + (void)startSession {
     }
 }
 
++ (void)stopSession {
+    if ([self bugsnagStarted]) {
+        [self.notifier stopSession];
+    }
+}
+
++ (BOOL)resumeSession {
+    if ([self bugsnagStarted]) {
+        return [self.notifier resumeSession];
+    } else {
+        return false;
+    }
+}
+
 + (NSDateFormatter *)payloadDateFormatter {
     static NSDateFormatter *formatter;
     static dispatch_once_t onceToken;

cocoa/vendor/bugsnag-cocoa/Source/BugsnagApiClient.h

@@ -7,7 +7,7 @@
 
 @class BugsnagConfiguration;
 
-typedef void (^RequestCompletion)(id data, BOOL success, NSError *error);
+typedef void (^RequestCompletion)(NSUInteger reportCount, BOOL success, NSError *error);
 
 @interface BugsnagApiClient : NSObject
 
@@ -21,11 +21,11 @@ typedef void (^RequestCompletion)(id data, BOOL success, NSError *error);
 
 - (NSOperation *)deliveryOperation;
 
-- (void)sendData:(id)data
-     withPayload:(NSDictionary *)payload
-           toURL:(NSURL *)url
-         headers:(NSDictionary *)headers
-    onCompletion:(RequestCompletion)onCompletion;
+- (void)sendItems:(NSUInteger)count
+      withPayload:(NSDictionary *)payload
+            toURL:(NSURL *)url
+          headers:(NSDictionary *)headers
+     onCompletion:(RequestCompletion)onCompletion;
 
 @property(readonly) NSOperationQueue *sendQueue;
 @property(readonly) BugsnagConfiguration *config;

cocoa/vendor/bugsnag-cocoa/Source/BugsnagApiClient.m

@@ -49,11 +49,11 @@ - (NSOperation *)deliveryOperation {
 #pragma mark - Delivery
 
 
-- (void)sendData:(id)data
-     withPayload:(NSDictionary *)payload
-           toURL:(NSURL *)url
-         headers:(NSDictionary *)headers
-    onCompletion:(RequestCompletion)onCompletion {
+- (void)sendItems:(NSUInteger)count
+      withPayload:(NSDictionary *)payload
+            toURL:(NSURL *)url
+          headers:(NSDictionary *)headers
+     onCompletion:(RequestCompletion)onCompletion {
 
     @try {
         NSError *error = nil;
@@ -64,7 +64,7 @@ - (void)sendData:(id)data
 
         if (jsonData == nil) {
             if (onCompletion) {
-                onCompletion(data, NO, error);
+                onCompletion(0, NO, error);
             }
             return;
         }
@@ -79,7 +79,7 @@ - (void)sendData:(id)data
                                 NSURLResponse *_Nullable response,
                                 NSError *_Nullable requestErr) {
                             if (onCompletion) {
-                                onCompletion(data, requestErr == nil, requestErr);
+                                onCompletion(count, requestErr == nil, requestErr);
                             }
                         }];
             [task resume];
@@ -92,13 +92,13 @@ - (void)sendData:(id)data
                                   returningResponse:&response
                                               error:&error];
             if (onCompletion) {
-                onCompletion(data, error == nil, error);
+                onCompletion(count, error == nil, error);
             }
 #pragma clang diagnostic pop
         }
     } @catch (NSException *exception) {
         if (onCompletion) {
-            onCompletion(data, NO,
+            onCompletion(count, NO,
                     [NSError            errorWithDomain:exception.reason
                                         code:420
                                     userInfo:@{BSGKeyException: exception}]);