Make entryPoint mandatory on SSOModal and check all fields on isSamlConfigured check

This commit is contained in:
carlosmonastyrski
2025-04-21 14:52:33 -03:00
parent 35f0e8f49a
commit 608e9a644c
2 changed files with 4 additions and 1 deletions
@@ -45,6 +45,7 @@ export const OrgAuthTab = withPermission(
const { data: samlConfig, isPending: isLoadingSamlConfig } = useGetSSOConfig( const { data: samlConfig, isPending: isLoadingSamlConfig } = useGetSSOConfig(
currentOrg?.id ?? "" currentOrg?.id ?? ""
); );
const { data: ldapConfig, isPending: isLoadingLdapConfig } = useGetLDAPConfig( const { data: ldapConfig, isPending: isLoadingLdapConfig } = useGetLDAPConfig(
currentOrg?.id ?? "" currentOrg?.id ?? ""
); );
@@ -54,7 +55,8 @@ export const OrgAuthTab = withPermission(
!enabledLoginMethods || enabledLoginMethods.includes(method); !enabledLoginMethods || enabledLoginMethods.includes(method);
const isOidcConfigured = oidcConfig && (oidcConfig.discoveryURL || oidcConfig.issuer); const isOidcConfigured = oidcConfig && (oidcConfig.discoveryURL || oidcConfig.issuer);
const isSamlConfigured = samlConfig && samlConfig.entryPoint; const isSamlConfigured =
samlConfig && (samlConfig.entryPoint || samlConfig.issuer || samlConfig.cert);
const isLdapConfigured = ldapConfig && ldapConfig.url; const isLdapConfigured = ldapConfig && ldapConfig.url;
const shouldShowCreateIdentityProviderView = const shouldShowCreateIdentityProviderView =
@@ -301,6 +301,7 @@ export const SSOModal = ({ popUp, handlePopUpClose, handlePopUpToggle, hideDelet
label={renderLabels(authProvider).entryPoint} label={renderLabels(authProvider).entryPoint}
errorText={error?.message} errorText={error?.message}
isError={Boolean(error)} isError={Boolean(error)}
isRequired
> >
<Input <Input
{...field} {...field}