fix: #995 and #1006, issues with redirect urls

Esri · Sep 1, 2022 · f03ed13 · f03ed13
1 parent 9227c4e
commit f03ed13
Show file tree

Hide file tree

Showing 6 changed files with 332 additions and 227 deletions.
diff --git a/demos/oauth2-browser/authenticate.html b/demos/oauth2-browser/authenticate.html
@@ -1,21 +1,25 @@
 <!DOCTYPE html>
 <html lang="en">
-  <head>
-    <meta charset="utf-8">
-    <title>ArcGIS Rest JS OAuth redirect</title>
-  </head>
-  <body>
-    <script src="@esri/arcgis-rest-request/dist/bundled/request.umd.js"></script>
-    <script src="config.js"></script>
-    <script>
-      let opts = {
-          clientId: config.clientId,
-          popup: true,
-        }
-      if (config.portal) {
-        opts.portal = config.portal
-      }
-      arcgisRest.ArcGISIdentityManager.completeOAuth2(opts);
-    </script>
-  </body>
-</html>
+
+<head>
+  <meta charset="utf-8">
+  <title>ArcGIS Rest JS OAuth redirect</title>
+</head>
+
+<body>
+  <script src="@esri/arcgis-rest-request/dist/bundled/request.umd.js"></script>
+  <script src="config.js"></script>
+  <script>
+    let opts = {
+      clientId: config.clientId,
+      redirectUri: config.popupRedirectUri,
+      popup: true,
+    }
+    if (config.portal) {
+      opts.portal = config.portal
+    }
+    arcgisRest.ArcGISIdentityManager.completeOAuth2(opts);
+  </script>
+</body>
+
+</html>