Merge pull request #1600 from microsoft/feature/track-error-null-check

Add null check for AppCenter.trackError
microsoft · Feb 25, 2022 · 578f9d9 · 578f9d9
2 parents 69d1214 + 7465ae0
commit 578f9d9
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,10 @@
 
 ## Version 4.4.3 (Under active development)
 
+### App Center Crashes
+
+* **[Fix]** Add exception null check for `Crashes.trackError` API.
+
 ### App Center Distribute
 
 * **[Fix]** Fix checking a new release if the application was already updated before.

diff --git a/sdk/appcenter-crashes/src/main/java/com/microsoft/appcenter/crashes/Crashes.java b/sdk/appcenter-crashes/src/main/java/com/microsoft/appcenter/crashes/Crashes.java
@@ -266,7 +266,7 @@ public static AppCenterFuture<Void> setEnabled(boolean enabled) {
      * @param throwable The throwable describing the handled error.
      */
     @SuppressWarnings({"SameParameterValue", "WeakerAccess"})
-    public static void trackError(Throwable throwable) {
+    public static void trackError(@NonNull Throwable throwable) {
         trackError(throwable, null, null);
     }
 
@@ -282,7 +282,7 @@ public static void trackError(Throwable throwable) {
      * @param properties  Optional properties.
      * @param attachments Optional attachments.
      */
-    public static void trackError(Throwable throwable, Map<String, String> properties, Iterable<ErrorAttachmentLog> attachments) {
+    public static void trackError(@NonNull Throwable throwable, Map<String, String> properties, Iterable<ErrorAttachmentLog> attachments) {
         getInstance().queueException(throwable, properties, attachments);
     }