Merge pull request #225 from fmartelli/master

[#224] fix
italia · Jul 15, 2022 · ea82f76 · ea82f76
2 parents bde4e9c + 3859cd5
commit ea82f76
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/spid-validator/server/lib/saml-utils.js b/spid-validator/server/lib/saml-utils.js
@@ -609,7 +609,16 @@ class IdP {
             qs += this.getLogoutResponsePayload(SAMLResponse, relayState, null);
         }
 
-        return url + "?" + qs;
+        const searchParams = new URLSearchParams(qs);
+
+        let slo = new URL(url);
+        let existingParams = new URLSearchParams(slo.search);
+        existingParams.forEach(function(value, key) {
+            searchParams.set(key, value);
+        });
+
+        slo.search = searchParams.toString();
+        return  slo.toString();
     }
 
     getLogoutResponsePayload(SAMLResponse, relayState, sigAlg) {