diff --git a/react-front-end/tsrc/settings/Integrations/oidc/OidcSettings.tsx b/react-front-end/tsrc/settings/Integrations/oidc/OidcSettings.tsx index 1e3e1fd9b2..082c95d416 100644 --- a/react-front-end/tsrc/settings/Integrations/oidc/OidcSettings.tsx +++ b/react-front-end/tsrc/settings/Integrations/oidc/OidcSettings.tsx @@ -317,7 +317,8 @@ const OidcSettings = ({ apiDetails, onApiDetailsChange, showValidationErrors, - serverHasConfiguration, + // OKTA platform doesn't have client secret field which means there is no secret configuration in the server. + serverHasConfiguration && initialIdpDetails.platform !== "OKTA", ); const handleOnSave = async () => { diff --git a/react-front-end/tsrc/settings/Integrations/oidc/OidcSettingsHelper.tsx b/react-front-end/tsrc/settings/Integrations/oidc/OidcSettingsHelper.tsx index 8056a57513..baf45865ee 100644 --- a/react-front-end/tsrc/settings/Integrations/oidc/OidcSettingsHelper.tsx +++ b/react-front-end/tsrc/settings/Integrations/oidc/OidcSettingsHelper.tsx @@ -309,7 +309,7 @@ const commonApiDetails = ( onChange: (key: string, value: unknown) => void, showValidationErrors: boolean, apiDetails: ApiDetails, - isConfigured: boolean, + isSecretConfigured: boolean, ): Record => { const { platform, apiUrl, apiClientId } = apiDetails; // apiClientSecret is not exist in OktaApiDetails. @@ -353,7 +353,7 @@ const commonApiDetails = ( desc: apiClientSecretDesc, required: true, // Validation is not required for updating but required for the initial creation. - validate: isConfigured ? constTrue : isNonEmptyString, + validate: isSecretConfigured ? constTrue : isNonEmptyString, component: passwordTextFiled({ name: apiClientSecretLabel, value: apiClientSecret, @@ -361,9 +361,9 @@ const commonApiDetails = ( required: true, onChange: (value) => onChange("apiClientSecret", value), showValidationErrors, - validate: isConfigured ? constTrue : isNonEmptyString, + validate: isSecretConfigured ? constTrue : isNonEmptyString, errorMessage: missingValue, - placeholder: isConfigured ? passwordMask : undefined, + placeholder: isSecretConfigured ? passwordMask : undefined, }), }, }; @@ -393,13 +393,13 @@ export const generatePlatform = ( * @param apiDetails The value of the platform specific details. * @param apiDetailsOnChange Function to be called when a platform specific field is changed. * @param showValidationErrors Whether to show validation errors for each field. - * @param isConfigured Whether the server already has the API details. + * @param isSecretConfigured Whether the server already has the API secrete. */ export const generateApiDetails = ( apiDetails: ApiDetails, apiDetailsOnChange: (key: string, value: unknown) => void, showValidationErrors: boolean, - isConfigured: boolean, + isSecretConfigured: boolean, ): Record => { const platform = apiDetails.platform; @@ -407,7 +407,7 @@ export const generateApiDetails = ( apiDetailsOnChange, showValidationErrors, apiDetails, - isConfigured, + isSecretConfigured, ); const { apiUrl, apiClientId, apiClientSecret } = apiCommonFields;