Added the metadata check, I think this will overlap with the current check, and should be done after, at other point of the app open process.

Author: gonzalonarbaiz

Examples/OneSignalDemo/local.properties

@@ -4,6 +4,6 @@
 # Location of the SDK. This is only used by Gradle.
 # For customization when using a Version Control System, please read the
 # header note.
-#Tue Jul 23 19:22:28 PDT 2019
+#Wed Jan 27 19:24:35 ART 2021
 ndk.dir=/Users/mikechoch/Library/Android/sdk/ndk-bundle
-sdk.dir=/Users/mikechoch/Library/Android/sdk
+sdk.dir=/Users/gonzalonarbaiz/Library/Android/sdk

OneSignalSDK/onesignal/src/main/java/com/onesignal/OSNotificationOpenedResult.java

@@ -58,6 +58,8 @@ public OSNotificationOpenedResult(OSNotification notification, OSNotificationAct
       // Configure 5 second timeout - max time we expect the application to call onResume
       // User can disable OneSignal application open by setting on the manifest:
       // <meta-data android:name="com.onesignal.NotificationOpened.DEFAULT" android:value="DISABLE" />
+      // User can also disable OneSignal from opening the url on default webview setting on the manifest:
+      // <meta-data android:name="com.onesignal.suppressLaunchURLs" android:value="true" />
       // If the user does this, we need to identify if the notification click was the way the user come back to the application (Session tracking)
       timeoutHandler = OSTimeoutHandler.getTimeoutHandler();
       timeoutRunnable = new Runnable() {

OneSignalSDK/onesignal/src/main/java/com/onesignal/OneSignal.java

@@ -2149,13 +2149,15 @@ public void run() {
 
       boolean urlOpened = false;
       boolean defaultOpenActionDisabled = "DISABLE".equals(OSUtils.getManifestMeta(context, "com.onesignal.NotificationOpened.DEFAULT"));
+      boolean launchUrlSuppress = "true".equals(OSUtils.getManifestMeta(context, "com.onesignal.suppressLaunchURLs"));
 
-      if (!defaultOpenActionDisabled)
+      if (!defaultOpenActionDisabled && !launchUrlSuppress)
          urlOpened = openURLFromNotification(context, data);
 
-      logger.debug("handleNotificationOpen from context: " + context + " with fromAlert: " + fromAlert + " urlOpened: " + urlOpened + " and defaultOpenActionDisabled: " + defaultOpenActionDisabled);
+      logger.debug("handleNotificationOpen from context: " + context + " with fromAlert: " + fromAlert + " urlOpened: " + urlOpened + " defaultOpenActionDisabled: " + defaultOpenActionDisabled
+              + " launchUrlSuppress: " + defaultOpenActionDisabled );
       // Check if the notification click should lead to a DIRECT session
-      if (shouldInitDirectSessionFromNotificationOpen(context, fromAlert, urlOpened, defaultOpenActionDisabled)) {
+      if (shouldInitDirectSessionFromNotificationOpen(context, fromAlert, urlOpened, defaultOpenActionDisabled, launchUrlSuppress)) {
          applicationOpenedByNotification(notificationId);
       }
 
@@ -2188,13 +2190,15 @@ static boolean startOrResumeApp(Activity inContext) {
     * 1. App is not an alert
     * 2. Not a URL open
     * 3. Manifest setting for com.onesignal.NotificationOpened.DEFAULT is not disabled
-    * 4. App is coming from the background
-    * 5. App open/resume intent exists
+    * 4. Manifest setting for com.onesignal.suppressLaunchURLs is not true
+    * 5. App is coming from the background
+    * 6. App open/resume intent exists
     */
-   private static boolean shouldInitDirectSessionFromNotificationOpen(Activity context, boolean fromAlert, boolean urlOpened, boolean defaultOpenActionDisabled) {
+   private static boolean shouldInitDirectSessionFromNotificationOpen(Activity context, boolean fromAlert, boolean urlOpened, boolean defaultOpenActionDisabled, boolean launchUrlSuppress) {
       return !fromAlert
               && !urlOpened
               && !defaultOpenActionDisabled
+              && !launchUrlSuppress
               && !inForeground
               && startOrResumeApp(context);
    }