fix response_mode defaulting logic

brockallen · brockallen · commit de5fd97d503e · 2018-12-09T12:21:36.000-05:00
diff --git a/src/OidcClient.js b/src/OidcClient.js
@@ -97,10 +97,9 @@ export class OidcClient {
     processSigninResponse(url, stateStore) {
         Log.debug("OidcClient.processSigninResponse");
 
-        let delimiter = "#";
-        if (this._settings.response_mode === "query" || SigninRequest.isCode(this._settings.response_type)) {
-            delimiter = "?";
-        }
+        let useQuery = this._settings.response_mode === "query" || 
+            (!this._settings.response_mode && SigninRequest.isCode(this._settings.response_type));
+        let delimiter = useQuery ? "?" : "#";
 
         var response = new SigninResponse(url, delimiter);
 
