Skip to content

Commit 629e943

Browse files
Fix client id is not populated to authflow session in saml flow
ref DEV-2350
2 parents 45d5dc0 + 2873d69 commit 629e943

File tree

3 files changed

+6
-5
lines changed

3 files changed

+6
-5
lines changed

.vettedpositions

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -41,10 +41,10 @@
4141
/pkg/auth/handler/webapp/auth_entry_point_middleware.go:31:31: requestcontext
4242
/pkg/auth/handler/webapp/auth_entry_point_middleware.go:32:35: requestcontext
4343
/pkg/auth/handler/webapp/authflow_change_password.go:96:26: requestcontext
44-
/pkg/auth/handler/webapp/authflow_controller.go:1000:19: requestcontext
45-
/pkg/auth/handler/webapp/authflow_controller.go:978:30: requestcontext
46-
/pkg/auth/handler/webapp/authflow_controller.go:983:24: requestcontext
47-
/pkg/auth/handler/webapp/authflow_controller.go:991:19: requestcontext
44+
/pkg/auth/handler/webapp/authflow_controller.go:1001:19: requestcontext
45+
/pkg/auth/handler/webapp/authflow_controller.go:979:30: requestcontext
46+
/pkg/auth/handler/webapp/authflow_controller.go:984:24: requestcontext
47+
/pkg/auth/handler/webapp/authflow_controller.go:992:19: requestcontext
4848
/pkg/auth/handler/webapp/authflow_create_password.go:132:26: requestcontext
4949
/pkg/auth/handler/webapp/authflow_enter_oob_otp.go:156:26: requestcontext
5050
/pkg/auth/handler/webapp/authflow_enter_password.go:138:26: requestcontext

pkg/auth/handler/webapp/authflow_controller.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -905,6 +905,7 @@ func (c *AuthflowController) makeSessionOptionsFromSAML(ctx context.Context, sam
905905
sessionOptions := &authflow.SessionOptions{
906906
SAMLSessionID: samlSessionID,
907907

908+
ClientID: uiInfo.SAMLServiceProviderID,
908909
RedirectURI: uiInfo.RedirectURI,
909910
Prompt: uiInfo.Prompt,
910911
LoginHint: uiInfo.LoginHint,

pkg/lib/saml/samlsession/ui.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ const (
1616
)
1717

1818
type SAMLUIInfo struct {
19-
// SAMLServiceProviderID is id of the service provider
19+
// SAMLServiceProviderID is id of the service provider, and also the corresponding client_id
2020
SAMLServiceProviderID string
2121
// RedirectURI is the redirect_uri the UI should redirect to.
2222
// The redirect_uri in the URL has lower precedence.

0 commit comments

Comments
 (0)