fix: duplicate URL Encode on RelayState (#29)

zitadel · Oct 24, 2022 · 80e4820 · 80e4820
1 parent c69137a
commit 80e4820
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/pkg/provider/logout_response.go b/pkg/provider/logout_response.go
@@ -6,7 +6,6 @@ import (
 	"encoding/base64"
 	"encoding/xml"
 	"net/http"
-	"net/url"
 	"text/template"
 	"time"
 
@@ -57,7 +56,7 @@ func (r *LogoutResponse) sendBackLogoutResponse(w http.ResponseWriter, resp *sam
 	samlMessage := base64.StdEncoding.EncodeToString(xmlbuff.Bytes())
 
 	data := LogoutResponseForm{
-		RelayState:   url.QueryEscape(r.RelayState),
+		RelayState:   r.RelayState,
 		SAMLResponse: samlMessage,
 		LogoutURL:    r.LogoutURL,
 	}

diff --git a/pkg/provider/response.go b/pkg/provider/response.go
@@ -4,7 +4,6 @@ import (
 	"encoding/base64"
 	"fmt"
 	"net/http"
-	"net/url"
 	"text/template"
 	"time"
 
@@ -55,7 +54,7 @@ func (r *Response) doResponse(request *http.Request, w http.ResponseWriter, resp
 		respData := base64.StdEncoding.EncodeToString([]byte(response))
 
 		data := AuthResponseForm{
-			url.QueryEscape(r.RelayState),
+			r.RelayState,
 			respData,
 			r.AcsUrl,
 		}