From 97b35561c15b712912e36c1ab7860a912bac3f61 Mon Sep 17 00:00:00 2001 From: Ron Date: Thu, 4 Apr 2024 10:53:22 -0400 Subject: [PATCH] User management connID and OrgID updates (#318) * user management connID and OrgID updates * updadating "TestClientAuthorizeURL" func --- pkg/usermanagement/client.go | 4 ++-- pkg/usermanagement/client_test.go | 10 +++++----- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/pkg/usermanagement/client.go b/pkg/usermanagement/client.go index fb638934..5afbfacd 100644 --- a/pkg/usermanagement/client.go +++ b/pkg/usermanagement/client.go @@ -741,10 +741,10 @@ func (c *Client) GetAuthorizationURL(opts GetAuthorizationURLOpts) (*url.URL, er query.Set("provider", string(opts.Provider)) } if opts.ConnectionID != "" { - query.Set("connection", opts.ConnectionID) + query.Set("connection_id", opts.ConnectionID) } if opts.OrganizationID != "" { - query.Set("organization", opts.OrganizationID) + query.Set("organization_id", opts.OrganizationID) } if opts.LoginHint != "" { query.Set("login_hint", opts.LoginHint) diff --git a/pkg/usermanagement/client_test.go b/pkg/usermanagement/client_test.go index 82fa94b4..ed500b7b 100644 --- a/pkg/usermanagement/client_test.go +++ b/pkg/usermanagement/client_test.go @@ -543,7 +543,7 @@ func TestClientAuthorizeURL(t *testing.T) { RedirectURI: "https://example.com/sso/workos/callback", State: "custom state", }, - expected: "https://api.workos.com/user_management/authorize?client_id=client_123&connection=connection_123&redirect_uri=https%3A%2F%2Fexample.com%2Fsso%2Fworkos%2Fcallback&response_type=code&state=custom+state", + expected: "https://api.workos.com/user_management/authorize?client_id=client_123&connection_id=connection_123&redirect_uri=https%3A%2F%2Fexample.com%2Fsso%2Fworkos%2Fcallback&response_type=code&state=custom+state", }, { scenario: "generate url with state", @@ -564,7 +564,7 @@ func TestClientAuthorizeURL(t *testing.T) { RedirectURI: "https://example.com/sso/workos/callback", State: "custom state", }, - expected: "https://api.workos.com/user_management/authorize?client_id=client_123&connection=connection_123&provider=GoogleOAuth&redirect_uri=https%3A%2F%2Fexample.com%2Fsso%2Fworkos%2Fcallback&response_type=code&state=custom+state", + expected: "https://api.workos.com/user_management/authorize?client_id=client_123&connection_id=connection_123&provider=GoogleOAuth&redirect_uri=https%3A%2F%2Fexample.com%2Fsso%2Fworkos%2Fcallback&response_type=code&state=custom+state", }, { scenario: "generate url with organization", @@ -574,7 +574,7 @@ func TestClientAuthorizeURL(t *testing.T) { RedirectURI: "https://example.com/sso/workos/callback", State: "custom state", }, - expected: "https://api.workos.com/user_management/authorize?client_id=client_123&organization=organization_123&redirect_uri=https%3A%2F%2Fexample.com%2Fsso%2Fworkos%2Fcallback&response_type=code&state=custom+state", + expected: "https://api.workos.com/user_management/authorize?client_id=client_123&organization_id=organization_123&redirect_uri=https%3A%2F%2Fexample.com%2Fsso%2Fworkos%2Fcallback&response_type=code&state=custom+state", }, { scenario: "generate url with DomainHint", @@ -585,7 +585,7 @@ func TestClientAuthorizeURL(t *testing.T) { State: "custom state", DomainHint: "foo.com", }, - expected: "https://api.workos.com/user_management/authorize?client_id=client_123&connection=connection_123&domain_hint=foo.com&redirect_uri=https%3A%2F%2Fexample.com%2Fsso%2Fworkos%2Fcallback&response_type=code&state=custom+state", + expected: "https://api.workos.com/user_management/authorize?client_id=client_123&connection_id=connection_123&domain_hint=foo.com&redirect_uri=https%3A%2F%2Fexample.com%2Fsso%2Fworkos%2Fcallback&response_type=code&state=custom+state", }, { scenario: "generate url with LoginHint", @@ -596,7 +596,7 @@ func TestClientAuthorizeURL(t *testing.T) { State: "custom state", LoginHint: "foo@workos.com", }, - expected: "https://api.workos.com/user_management/authorize?client_id=client_123&connection=connection_123&login_hint=foo%40workos.com&redirect_uri=https%3A%2F%2Fexample.com%2Fsso%2Fworkos%2Fcallback&response_type=code&state=custom+state", + expected: "https://api.workos.com/user_management/authorize?client_id=client_123&connection_id=connection_123&login_hint=foo%40workos.com&redirect_uri=https%3A%2F%2Fexample.com%2Fsso%2Fworkos%2Fcallback&response_type=code&state=custom+state", }, }