fix: Don't attach Android Threads (#866)

Author: bruno-garcia

android/src/main/java/io/sentry/RNSentryModule.java

@@ -64,7 +64,9 @@ public Map<String, Object> getConstants() {
     public void startWithOptions(final ReadableMap rnOptions, Promise promise) {
         SentryAndroid.init(this.getReactApplicationContext(), options -> {
             if (rnOptions.hasKey("dsn") && rnOptions.getString("dsn") != null) {
-                options.setDsn(rnOptions.getString("dsn"));
+                String dsn = rnOptions.getString("dsn");
+                logger.info(String.format("Starting with DSN: '%s'", dsn));
+                options.setDsn(dsn);
             } else {
                 // SentryAndroid needs an empty string fallback for the dsn.
                 options.setDsn("");
@@ -82,6 +84,10 @@ public void startWithOptions(final ReadableMap rnOptions, Promise promise) {
                 options.setDist(rnOptions.getString("dist"));
             }
 
+            // JS use top level stacktraces and android attaches Threads which hides them so by default we hide.
+            boolean attachThreads = rnOptions.hasKey("attachThreads") && rnOptions.getBoolean("attachThreads");
+            options.setAttachThreads(attachThreads);
+
             options.setBeforeSend((event, hint) -> {
                 // React native internally throws a JavascriptException
                 // Since we catch it before that, we don't want to send this one
@@ -112,10 +118,6 @@ public void startWithOptions(final ReadableMap rnOptions, Promise promise) {
             sentryOptions = options;
         });
 
-        if (rnOptions.hasKey("dsn")) {
-            logger.info(String.format("startWithDsnString '%s'", rnOptions.getString("dsn")));
-        }
-
         promise.resolve(true);
     }