Add an api call for web config if no default url

There was an issue with legacy HTTP integrations where the
defaultNotificationUrl was never saved in the DB. To account for this,
we try to get the default URL from the app config on notification click
and save it to the DB for the future use.

Choosing between notification received and clicked to add this logic,
decided on notification clicked to avoid doing extra api call when the
user may never click the notification.

Author: itrush

src/service-worker/ServiceWorker.ts

@@ -28,6 +28,7 @@ import { NotificationReceived, NotificationClicked } from "../models/Notificatio
 import { cancelableTimeout } from "../helpers/sw/CancelableTimeout";
 import { DeviceRecord } from '../models/DeviceRecord';
 import { awaitableTimeout } from "../utils/AwaitableTimeout";
+import { AppConfig } from "src/models/AppConfig";
 
 declare var self: ServiceWorkerGlobalScope & OSServiceWorkerFields;
 
@@ -156,6 +157,11 @@ export class ServiceWorker {
     return appId;
   }
 
+  static async getAppConfig(): Promise<AppConfig> {
+    const appId = await ServiceWorker.getAppId();
+    return await ConfigHelper.getAppConfig({ appId }, OneSignalApiSW.downloadServerAppConfig);
+  }
+
   static setupMessageListeners() {
     ServiceWorker.workerMessenger.on(WorkerMessengerCommand.WorkerVersion, _ => {
       Log.debug('[Service Worker] Received worker version message.');
@@ -750,8 +756,33 @@ export class ServiceWorker {
 
     // Use the user-provided default URL if one exists
     const { defaultNotificationUrl: dbDefaultNotificationUrl } = await Database.getAppState();
-    if (dbDefaultNotificationUrl)
+    if (dbDefaultNotificationUrl) {
       launchUrl = dbDefaultNotificationUrl;
+    }
+    else {
+      // There was an issue with legacy HTTP integrations where the defaultNotificationUrl
+      // was never saved to the DB. To account for this, we try to get the default URL
+      // from the app config on notification click and save it to the DB for future use.
+      // Choosing between notification received and notification clicked to do this logic,
+      // we decided on notification clicked to avoid doing extra api call when the user
+      // may never click the notification.
+      try {
+        const config = await ServiceWorker.getAppConfig();
+        const defaultNotificationUrlFromConfig = config.origin;
+        if (defaultNotificationUrlFromConfig) {
+          launchUrl = defaultNotificationUrlFromConfig;
+        }
+      } catch (e) {
+        Log.error("Failed to get app config to determine default notification url", e);
+      }
+
+      if (!!launchUrl && !dbDefaultNotificationUrl) {
+        // intentionally not awaiting this promise to not block notification click handling
+        Database.setAppState({ defaultNotificationUrl: launchUrl }).catch(e => {
+          Log.error("Failed to save default notification url to db", e);
+        });
+      }
+    }
 
     // If the user clicked an action button, use the URL provided by the action button
     // Unless the action button URL is null