Skip to content

Commit 4548688

Browse files
authored
fix(console): drop empty entries (#7935)
1 parent cc067e1 commit 4548688

File tree

2 files changed

+9
-2
lines changed

2 files changed

+9
-2
lines changed

packages/console/src/pages/SignInExperience/PageContent/index.tsx

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ import {
3030
type SignInExperiencePageManagedData,
3131
type SignInExperienceForm,
3232
type AccountCenterFormValues,
33+
normalizeWebauthnRelatedOrigins,
3334
} from '../types';
3435

3536
import AccountCenter from './AccountCenter';
@@ -97,6 +98,9 @@ function PageContent({ data, onSignInExperienceUpdated, onAccountCenterUpdated }
9798

9899
try {
99100
const { accountCenter, ...formValues } = getValues();
101+
const webauthnRelatedOrigins = normalizeWebauthnRelatedOrigins(
102+
accountCenter.webauthnRelatedOrigins
103+
);
100104

101105
const updatedData = await api
102106
.patch('api/sign-in-exp', {
@@ -110,7 +114,7 @@ function PageContent({ data, onSignInExperienceUpdated, onAccountCenterUpdated }
110114
enabled: accountCenter.enabled,
111115
// Disable all fields when account center is disabled
112116
fields: accountCenter.enabled ? accountCenter.fields : {},
113-
webauthnRelatedOrigins: accountCenter.webauthnRelatedOrigins,
117+
webauthnRelatedOrigins,
114118
},
115119
})
116120
.json<AccountCenterConfig>();

packages/console/src/pages/SignInExperience/types.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,9 @@ const createDefaultAccountCenterFormValues = (): AccountCenterFormValues => ({
5555
webauthnRelatedOrigins: [],
5656
});
5757

58+
export const normalizeWebauthnRelatedOrigins = (origins?: string[]): string[] =>
59+
origins?.map((origin) => origin.trim()).filter(Boolean) ?? [];
60+
5861
export const convertAccountCenterToForm = (
5962
accountCenter?: AccountCenterConfig
6063
): AccountCenterFormValues => ({
@@ -63,7 +66,7 @@ export const convertAccountCenterToForm = (
6366
...createDefaultAccountCenterFormValues().fields,
6467
...accountCenter?.fields,
6568
},
66-
webauthnRelatedOrigins: accountCenter?.webauthnRelatedOrigins ?? [],
69+
webauthnRelatedOrigins: normalizeWebauthnRelatedOrigins(accountCenter?.webauthnRelatedOrigins),
6770
});
6871

6972
/**

0 commit comments

Comments
 (0)