Skip to content

Commit f4ac74b

Browse files
authored
fix: fix SAML app console issues (#6970)
1 parent 2633723 commit f4ac74b

File tree

19 files changed

+71
-51
lines changed

19 files changed

+71
-51
lines changed
Loading

packages/console/src/pages/ApplicationDetails/SamlApplicationDetailsContent/CertificateActionMenu/index.tsx

+28-32
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,9 @@ import { useTranslation } from 'react-i18next';
44

55
import Delete from '@/assets/icons/delete.svg?react';
66
import Download from '@/assets/icons/download.svg?react';
7-
import Deactivate from '@/assets/icons/moon.svg?react';
7+
import Deactivate from '@/assets/icons/forbidden.svg?react';
88
import More from '@/assets/icons/more.svg?react';
9-
import Activate from '@/assets/icons/sun.svg?react';
9+
import Activate from '@/assets/icons/shield.svg?react';
1010
import ActionMenu, { ActionMenuItem } from '@/ds-components/ActionMenu';
1111
import { downloadText } from '@/utils/downloader';
1212

@@ -41,42 +41,38 @@ function CertificateActionMenu({
4141

4242
return (
4343
<ActionMenu icon={<More className={styles.icon} />} title={t('general.more_options')}>
44-
{active ? (
44+
<ActionMenuItem
45+
iconClassName={styles.icon}
46+
icon={active ? <Deactivate /> : <Activate />}
47+
onClick={() => {
48+
if (active) {
49+
onDeactivate(id);
50+
} else {
51+
onActivate(id);
52+
}
53+
}}
54+
>
55+
{t(`general.${active ? 'deactivate' : 'activate'}`)}
56+
</ActionMenuItem>
57+
<ActionMenuItem
58+
iconClassName={styles.icon}
59+
icon={<Download className={styles.icon} />}
60+
onClick={onDownload}
61+
>
62+
{t('general.download')}
63+
</ActionMenuItem>
64+
{!active && (
65+
// Can only delete inactive certificates.
4566
<ActionMenuItem
46-
iconClassName={styles.icon}
47-
icon={<Deactivate />}
67+
type="danger"
68+
icon={<Delete />}
4869
onClick={() => {
49-
onDeactivate(id);
70+
onDelete(id);
5071
}}
5172
>
52-
{t('general.deactivate')}
73+
{t('general.delete')}
5374
</ActionMenuItem>
54-
) : (
55-
<>
56-
{/* Can only delete inactive certificates */}
57-
<ActionMenuItem
58-
type="danger"
59-
icon={<Delete />}
60-
onClick={() => {
61-
onDelete(id);
62-
}}
63-
>
64-
{t('general.delete')}
65-
</ActionMenuItem>
66-
<ActionMenuItem
67-
iconClassName={styles.icon}
68-
icon={<Activate />}
69-
onClick={() => {
70-
onActivate(id);
71-
}}
72-
>
73-
{t('general.activate')}
74-
</ActionMenuItem>
75-
</>
7675
)}
77-
<ActionMenuItem iconClassName={styles.icon} icon={<Download />} onClick={onDownload}>
78-
{t('general.download')}
79-
</ActionMenuItem>
8076
</ActionMenu>
8177
);
8278
}

packages/console/src/pages/ApplicationDetails/SamlApplicationDetailsContent/Settings.tsx

+8-4
Original file line numberDiff line numberDiff line change
@@ -208,18 +208,22 @@ function Settings({ data, mutateApplication, isDeleted }: Props) {
208208
!value || uriValidator(value) || t('errors.invalid_uri_format'),
209209
})}
210210
error={Boolean(errors.acsUrl)}
211-
placeholder={t('enterprise_sso.basic_info.saml.acs_url_field_name')}
211+
placeholder={t('enterprise_sso.basic_info.saml.acs_url_field_placeholder')}
212212
/>
213213
</FormField>
214-
<FormField isRequired title="enterprise_sso.basic_info.saml.audience_uri_field_name">
214+
<FormField
215+
isRequired
216+
title="enterprise_sso.basic_info.saml.entity_id_field_name"
217+
tip={t('enterprise_sso.basic_info.saml.entity_id_field_tooltip')}
218+
>
215219
<TextInput
216220
{...register('entityId', {
217221
required: t('errors.required_field_missing', {
218-
field: t('enterprise_sso.basic_info.saml.audience_uri_field_name'),
222+
field: t('enterprise_sso.basic_info.saml.entity_id_field_name'),
219223
}),
220224
})}
221225
error={Boolean(errors.entityId)}
222-
placeholder={t('enterprise_sso.basic_info.saml.audience_uri_field_name')}
226+
placeholder={t('enterprise_sso.basic_info.saml.entity_id_field_placeholder')}
223227
/>
224228
</FormField>
225229
</FormCard>

packages/phrases/src/locales/ar/translation/admin-console/enterprise-sso.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,8 @@ const enterprise_sso = {
2626
'قم بإنشاء تكامل تطبيق جديد باستخدام SAML 2.0 في موفر الهوية {{name}} الخاص بك. ثم قم بلصق القيمة التالية فيه.',
2727
saml: {
2828
acs_url_field_name: 'عنوان URL لخدمة المستهلك للتأكيد (رد URL)',
29-
audience_uri_field_name: 'معرف الجمهور (معرف SP)',
29+
/** UNTRANSLATED */
30+
audience_uri_field_name: 'Audience URI (SP Entity ID)',
3031
},
3132
oidc: {
3233
redirect_uri_field_name: 'عنوان URL لإعادة التوجيه (عنوان URL للرد)',

packages/phrases/src/locales/de/translation/admin-console/enterprise-sso.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,8 @@ const enterprise_sso = {
2727
'Erstellen Sie eine neue Anwendungsintegration per SAML 2.0 in Ihrem {{name}}-Identitätsanbieter. Fügen Sie dann den folgenden Wert hinzu.',
2828
saml: {
2929
acs_url_field_name: 'Zieldienst-URL für Assertionsverbrauch (Antwort-URL)',
30-
audience_uri_field_name: 'Ziel-URI (SP Entity ID)',
30+
/** UNTRANSLATED */
31+
audience_uri_field_name: 'Audience URI (SP Entity ID)',
3132
},
3233
oidc: {
3334
redirect_uri_field_name: 'Weiterleitungs-URI (Callback-URL)',

packages/phrases/src/locales/en/translation/admin-console/enterprise-sso.ts

+5
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,11 @@ const enterprise_sso = {
2828
saml: {
2929
acs_url_field_name: 'Assertion consumer service URL (Reply URL)',
3030
audience_uri_field_name: 'Audience URI (SP Entity ID)',
31+
entity_id_field_name: 'Service Provider (SP) Entity ID',
32+
entity_id_field_tooltip:
33+
'The SP Entity ID can be in any string format, typically using a URI form or a URL form as an identifier, but this is not mandatory.',
34+
acs_url_field_placeholder: 'https://your-domain.com/api/saml/callback',
35+
entity_id_field_placeholder: 'urn:your-domain.com:sp:saml:{serviceProviderId}',
3136
},
3237
oidc: {
3338
redirect_uri_field_name: 'Redirect URI (Callback URL)',

packages/phrases/src/locales/es/translation/admin-console/enterprise-sso.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,8 @@ const enterprise_sso = {
2828
'Crea una nueva integración de aplicación mediante SAML 2.0 en tu proveedor de identidad {{name}}. Luego pega el siguiente valor en él.',
2929
saml: {
3030
acs_url_field_name: 'URL de servicio de consumidor de aserciones (URL de respuesta)',
31-
audience_uri_field_name: 'URI del público (ID del SP entidad)',
31+
/** UNTRANSLATED */
32+
audience_uri_field_name: 'Audience URI (SP Entity ID)',
3233
},
3334
oidc: {
3435
redirect_uri_field_name: 'URI de redireccionamiento (URL de devolución de llamada)',

packages/phrases/src/locales/fr/translation/admin-console/enterprise-sso.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,8 @@ const enterprise_sso = {
2727
"Créez une nouvelle intégration d'application par SAML 2.0 dans votre fournisseur d'identité {{name}}. Ensuite, collez la valeur suivante.",
2828
saml: {
2929
acs_url_field_name: "URL de service de consommation d'assertions (URL de réponse)",
30-
audience_uri_field_name: "URI de l'audience (ID de l'entité SP)",
30+
/** UNTRANSLATED */
31+
audience_uri_field_name: 'Audience URI (SP Entity ID)',
3132
},
3233
oidc: {
3334
redirect_uri_field_name: 'URI de redirection (URL de rappel)',

packages/phrases/src/locales/it/translation/admin-console/enterprise-sso.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,8 @@ const enterprise_sso = {
2727
"Crea una nuova integrazione dell'applicazione tramite SAML 2.0 nel tuo provider di identità {{name}}. Quindi incolla il valore seguente.",
2828
saml: {
2929
acs_url_field_name: 'URL del servizio consumer di asserzioni (URL di risposta)',
30-
audience_uri_field_name: "URI dell'audience (ID entità SP)",
30+
/** UNTRANSLATED */
31+
audience_uri_field_name: 'Audience URI (SP Entity ID)',
3132
},
3233
oidc: {
3334
redirect_uri_field_name: 'URI di reindirizzamento (URL di callback)',

packages/phrases/src/locales/ja/translation/admin-console/enterprise-sso.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,8 @@ const enterprise_sso = {
2727
'IdPにてSAML 2.0の新しいアプリケーション連携を作成します。次に以下の値を貼り付けます。',
2828
saml: {
2929
acs_url_field_name: 'アサーションコンシューマサービスURL(応答URL)',
30-
audience_uri_field_name: 'オーディエンスURI(SPエンティティID)',
30+
/** UNTRANSLATED */
31+
audience_uri_field_name: 'Audience URI (SP Entity ID)',
3132
},
3233
oidc: {
3334
redirect_uri_field_name: 'リダイレクトURI(コールバックURL)',

packages/phrases/src/locales/ko/translation/admin-console/enterprise-sso.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,8 @@ const enterprise_sso = {
2626
'SAML 2.0을 사용하여 {{name}} 신원 공급자에서 새로운 응용 프로그램 통합을 만듭니다. 그런 다음 다음 값을 붙여 넣습니다.',
2727
saml: {
2828
acs_url_field_name: '단언 소비자 서비스 URL (응답 URL)',
29-
audience_uri_field_name: '대상 URI (SP 엔터티 ID)',
29+
/** UNTRANSLATED */
30+
audience_uri_field_name: 'Audience URI (SP Entity ID)',
3031
},
3132
oidc: {
3233
redirect_uri_field_name: '리디렉션 URI (콜백 URL)',

packages/phrases/src/locales/pl-pl/translation/admin-console/enterprise-sso.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,8 @@ const enterprise_sso = {
2727
'Utwórz nową integrację aplikacji za pomocą SAML 2.0 w dostawcy tożsamości {{name}}. Następnie wklej poniższą wartość.',
2828
saml: {
2929
acs_url_field_name: 'Adres URL usługi konsumenta twierdzeń (Adres URL odpowiedzi)',
30-
audience_uri_field_name: 'URI odbiorcy (ID podmiotu SP)',
30+
/** UNTRANSLATED */
31+
audience_uri_field_name: 'Audience URI (SP Entity ID)',
3132
},
3233
oidc: {
3334
redirect_uri_field_name: 'Adres URL przekierowania (Adres URL wywołania zwrotnego)',

packages/phrases/src/locales/pt-br/translation/admin-console/enterprise-sso.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,8 @@ const enterprise_sso = {
2828
'Crie uma nova integração de aplicativo por SAML 2.0 em seu provedor de identidade {{name}}. Em seguida, cole o seguinte valor nele.',
2929
saml: {
3030
acs_url_field_name: 'URL do serviço de consumidor de afirmações (URL de resposta)',
31-
audience_uri_field_name: 'URI da audiência (ID da Entidade do SP)',
31+
/** UNTRANSLATED */
32+
audience_uri_field_name: 'Audience URI (SP Entity ID)',
3233
},
3334
oidc: {
3435
redirect_uri_field_name: 'URL de redirecionamento (URL de retorno de chamada)',

packages/phrases/src/locales/pt-pt/translation/admin-console/enterprise-sso.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,8 @@ const enterprise_sso = {
2727
'Crie uma nova integração de aplicativo usando SAML 2.0 no seu provedor de identidade {{name}}. Em seguida, cole o seguinte valor nele.',
2828
saml: {
2929
acs_url_field_name: 'URL do serviço de consumo de afirmação (URL de resposta)',
30-
audience_uri_field_name: 'URI do público (ID da entidade do SP)',
30+
/** UNTRANSLATED */
31+
audience_uri_field_name: 'Audience URI (SP Entity ID)',
3132
},
3233
oidc: {
3334
redirect_uri_field_name: 'URI de redirecionamento (URL de retorno)',

packages/phrases/src/locales/ru/translation/admin-console/enterprise-sso.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,8 @@ const enterprise_sso = {
2727
'Создайте новую интеграцию приложения с помощью SAML 2.0 в вашем провайдере идентичности {{name}}. Затем вставьте следующее значение в него.',
2828
saml: {
2929
acs_url_field_name: 'URL потребителя утверждений (URL ответа)',
30-
audience_uri_field_name: 'Адрес аудиенции (SP Entity ID)',
30+
/** UNTRANSLATED */
31+
audience_uri_field_name: 'Audience URI (SP Entity ID)',
3132
},
3233
oidc: {
3334
redirect_uri_field_name: 'URI перенаправления (URL обратного вызова)',

packages/phrases/src/locales/tr-tr/translation/admin-console/enterprise-sso.ts

+1
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@ const enterprise_sso = {
2727
'Yeni bir uygulama entegrasyonu oluşturun, {{name}} kimlik sağlayıcınızda SAML 2.0 ile. Ardından aşağıdaki değeri yapıştırın.',
2828
saml: {
2929
acs_url_field_name: 'Assertion consumer service URL (Reply URL)',
30+
/** UNTRANSLATED */
3031
audience_uri_field_name: 'Audience URI (SP Entity ID)',
3132
},
3233
oidc: {

packages/phrases/src/locales/zh-cn/translation/admin-console/enterprise-sso.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,8 @@ const enterprise_sso = {
2525
description: '在{{name}}身份提供者中通过SAML 2.0创建一个新的应用集成。然后将以下值粘贴到其中。',
2626
saml: {
2727
acs_url_field_name: '断言消费者服务URL(回复URL)',
28-
audience_uri_field_name: '受众URI(SP实体ID)',
28+
/** UNTRANSLATED */
29+
audience_uri_field_name: 'Audience URI (SP Entity ID)',
2930
},
3031
oidc: {
3132
redirect_uri_field_name: '重定向URI(回调URL)',

packages/phrases/src/locales/zh-hk/translation/admin-console/enterprise-sso.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,8 @@ const enterprise_sso = {
2525
description: '在您的{{name}}身份提供者中通過 SAML 2.0 創建新應用集成。然後將以下值貼到其中。',
2626
saml: {
2727
acs_url_field_name: '斷言消費服務 URL (回複 URL)',
28-
audience_uri_field_name: '受眾 URI (SP 實體 ID)',
28+
/** UNTRANSLATED */
29+
audience_uri_field_name: 'Audience URI (SP Entity ID)',
2930
},
3031
oidc: {
3132
redirect_uri_field_name: '重定向 URI(回調 URL)',

packages/phrases/src/locales/zh-tw/translation/admin-console/enterprise-sso.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,8 @@ const enterprise_sso = {
2626
'在您的 {{name}} 身份提供者中使用 SAML 2.0 建立新的應用集成。然後將以下值粘貼到其中。',
2727
saml: {
2828
acs_url_field_name: '斷言消費服務 URL(回覆 URL)',
29-
audience_uri_field_name: '受眾 URI(SP 實體 ID)',
29+
/** UNTRANSLATED */
30+
audience_uri_field_name: 'Audience URI (SP Entity ID)',
3031
},
3132
oidc: {
3233
redirect_uri_field_name: '重定向 URI(回撥 URL)',

0 commit comments

Comments
 (0)