Merge pull request #1258 from m-mohr/patch-1

Fix #1256: AuthCode w/ PKCE not supported for popups
DuendeArchive · Jan 30, 2021 · 2be23d0 · 2be23d0
2 parents d5f5789 + 047f9b9
commit 2be23d0
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/UserManager.js b/src/UserManager.js
@@ -424,7 +424,9 @@ export class UserManager extends OidcClient {
     }
     _signinCallback(url, navigator) {
         Log.debug("UserManager._signinCallback");
-        return navigator.callback(url);
+        let useQuery = this._settings.response_mode === "query" || (!this._settings.response_mode && SigninRequest.isCode(this._settings.response_type));
+        let delimiter = useQuery ? "?" : "#";
+        return navigator.callback(url, undefined, delimiter);
     }
 
     signoutRedirect(args = {}) {