Skip to content

Commit 5e6dd29

Browse files
committed
use query sign in redirect
1 parent fcc2cc5 commit 5e6dd29

File tree

1 file changed

+4
-11
lines changed

1 file changed

+4
-11
lines changed

service_provider.go

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -271,19 +271,12 @@ func (req *AuthnRequest) Redirect(relayState string, sp *ServiceProvider) (*url.
271271
query += "&RelayState=" + relayState
272272
}
273273
if len(sp.SignatureMethod) > 0 {
274-
sp.signQuery(samlRequest, query, reqString, relayState)
275-
query += "&SigAlg=" + url.QueryEscape(sp.SignatureMethod)
276-
signingContext, err := GetSigningContext(sp)
277-
278-
if err != nil {
279-
return nil, err
274+
var errSig error
275+
query, errSig = sp.signQuery(samlRequest, query, reqString, relayState)
276+
if errSig != nil {
277+
return nil, errSig
280278
}
281279

282-
sig, err := signingContext.SignString(query)
283-
if err != nil {
284-
return nil, err
285-
}
286-
query += "&Signature=" + url.QueryEscape(base64.StdEncoding.EncodeToString(sig))
287280
}
288281

289282
rv.RawQuery = query

0 commit comments

Comments
 (0)