change dsn logic

Author: AbhiPrasad

packages/browser/src/sdk.ts

@@ -135,12 +135,13 @@ export function showReportDialog(options: ReportDialogOptions = {}, hub: Hub = g
     return;
   }
 
-  if (!options.dsn) {
-    IS_DEBUG_BUILD && logger.error('Missing dsn option in showReportDialog call');
+  const { client, scope } = hub.getStackTop();
+  const dsn = options.dsn || (client && client.getDsn());
+  if (!dsn) {
+    IS_DEBUG_BUILD && logger.error('DSN not configured for showReportDialog call');
     return;
   }
 
-  const scope = hub.getScope();
   if (scope) {
     options.user = {
       ...scope.getUser(),
@@ -154,7 +155,7 @@ export function showReportDialog(options: ReportDialogOptions = {}, hub: Hub = g
 
   const script = global.document.createElement('script');
   script.async = true;
-  script.src = getReportDialogEndpoint(options.dsn, options);
+  script.src = getReportDialogEndpoint(dsn, options);
 
   if (options.onLoad) {
     // eslint-disable-next-line @typescript-eslint/unbound-method

packages/core/src/api.ts

@@ -118,14 +118,15 @@ export function getReportDialogEndpoint(
     }
 
     if (key === 'user') {
-      if (!dialogOptions.user) {
+      const user = dialogOptions.user;
+      if (!user) {
         continue;
       }
-      if (dialogOptions.user.name) {
-        encodedOptions += `&name=${encodeURIComponent(dialogOptions.user.name)}`;
+      if (user.name) {
+        encodedOptions += `&name=${encodeURIComponent(user.name)}`;
       }
-      if (dialogOptions.user.email) {
-        encodedOptions += `&email=${encodeURIComponent(dialogOptions.user.email)}`;
+      if (user.email) {
+        encodedOptions += `&email=${encodeURIComponent(user.email)}`;
       }
     } else {
       encodedOptions += `&${encodeURIComponent(key)}=${encodeURIComponent(dialogOptions[key] as string)}`;