Skip to content

Commit

Permalink
fix: remove redirecturl from base SSO config
Browse files Browse the repository at this point in the history
  • Loading branch information
jrea committed Jul 28, 2023
1 parent b7e8a69 commit 037395e
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 18 deletions.
11 changes: 0 additions & 11 deletions packages/react/src/SSO/BaseSSOForm.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -69,16 +69,6 @@ export default function BaseSSOForm(
required: true,
disabled: !allowEdit,
},
{
name: 'redirectURI',
label: 'Redirect URI',
type: AttributeType.Text,
helpText:
'Where users should be redirected to after a successful login',
defaultValue: config?.redirectURI ?? '',
required: true,
disabled: !allowEdit,
},
{
name: 'emailDomains',
label: 'Email domains',
Expand Down Expand Up @@ -107,7 +97,6 @@ export default function BaseSSOForm(
config?.configUrl,
config?.emailDomains,
config?.enabled,
config?.redirectURI,
]);

const mutation = useMutation(
Expand Down
3 changes: 0 additions & 3 deletions packages/react/test/SSO/Okta.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -34,9 +34,6 @@ describe('Okta', () => {
const configUrl = screen.getByLabelText('Config url');
fireEvent.change(configUrl, { target: { value: 'configUrl' } });

const redirectURI = screen.getByLabelText('Redirect URI');
fireEvent.change(redirectURI, { target: { value: 'redirectURI' } });

const emailDomains = screen.getByLabelText('Email domains');
fireEvent.change(emailDomains, { target: { value: 'emailDomains' } });

Expand Down
4 changes: 0 additions & 4 deletions packages/server/openapi/index.json
Original file line number Diff line number Diff line change
Expand Up @@ -233,10 +233,6 @@
"clientSecret": {
"type": "string"
},
"redirectURI": {
"type": "string",
"format": "uri"
},
"emailDomains": {
"type": "array",
"items": {
Expand Down

0 comments on commit 037395e

Please sign in to comment.