Commit aa98f798 authored by Peter Hegman's avatar Peter Hegman

Merge branch 'alvin-sso-enforcement-text-update' into 'master'

Enhance SSO enforcement warning message

See merge request gitlab-org/gitlab!73233
parents 9a0a57c7 d8c31664
...@@ -15,7 +15,7 @@ ...@@ -15,7 +15,7 @@
= s_('GroupSAML|Before enforcing SSO, enable SAML authentication.') = s_('GroupSAML|Before enforcing SSO, enable SAML authentication.')
%p.help-text.js-warning{ class: saml_provider.enforced_sso? && 'gl-display-none' } %p.help-text.js-warning{ class: saml_provider.enforced_sso? && 'gl-display-none' }
- link_start = '<a href="%{url}" target="_blank" rel="noopener noreferrer">'.html_safe % { url: help_page_path('user/group/saml_sso/index', anchor: 'sso-enforcement') } - link_start = '<a href="%{url}" target="_blank" rel="noopener noreferrer">'.html_safe % { url: help_page_path('user/group/saml_sso/index', anchor: 'sso-enforcement') }
= html_escape(s_('GroupSAML|%{strongOpen}Warning%{strongClose} - Enabling %{linkStart}SSO enforcement%{linkEnd} can reduce security risks.')) % { strongOpen: '<strong>'.html_safe, strongClose: '</strong>'.html_safe, linkStart: link_start, linkEnd: '</a>'.html_safe } = html_escape(s_('GroupSAML|%{strongOpen}Warning%{strongClose} - Enable %{linkStart}SSO enforcement%{linkEnd} to reduce security risks.')) % { strongOpen: '<strong>'.html_safe, strongClose: '</strong>'.html_safe, linkStart: link_start, linkEnd: '</a>'.html_safe }
.form-group .form-group
.gl-form-checkbox.custom-control.custom-checkbox .gl-form-checkbox.custom-control.custom-checkbox
= f.check_box :git_check_enforced, { class: 'custom-control-input js-group-saml-enforced-git-check-input' } = f.check_box :git_check_enforced, { class: 'custom-control-input js-group-saml-enforced-git-check-input' }
......
...@@ -71,7 +71,7 @@ RSpec.describe 'SAML provider settings' do ...@@ -71,7 +71,7 @@ RSpec.describe 'SAML provider settings' do
uncheck 'Enforce SSO-only authentication for web activity for this group' uncheck 'Enforce SSO-only authentication for web activity for this group'
expect(page).to have_content 'Warning - Enabling SSO enforcement can reduce security risks.' expect(page).to have_content 'Warning - Enable SSO enforcement to reduce security risks.'
end end
end end
...@@ -118,7 +118,7 @@ RSpec.describe 'SAML provider settings' do ...@@ -118,7 +118,7 @@ RSpec.describe 'SAML provider settings' do
uncheck 'Enforce SSO-only authentication for web activity for this group' uncheck 'Enforce SSO-only authentication for web activity for this group'
expect { submit }.to change { saml_provider.reload.enforced_sso }.to(false) expect { submit }.to change { saml_provider.reload.enforced_sso }.to(false)
expect(page).to have_content 'Warning - Enabling SSO enforcement can reduce security risks.' expect(page).to have_content 'Warning - Enable SSO enforcement to reduce security risks.'
end end
context 'enforced_group_managed_accounts enabled', :js do context 'enforced_group_managed_accounts enabled', :js do
......
...@@ -16495,7 +16495,7 @@ msgstr "" ...@@ -16495,7 +16495,7 @@ msgstr ""
msgid "GroupSAML|\"persistent\" recommended" msgid "GroupSAML|\"persistent\" recommended"
msgstr "" msgstr ""
msgid "GroupSAML|%{strongOpen}Warning%{strongClose} - Enabling %{linkStart}SSO enforcement%{linkEnd} can reduce security risks." msgid "GroupSAML|%{strongOpen}Warning%{strongClose} - Enable %{linkStart}SSO enforcement%{linkEnd} to reduce security risks."
msgstr "" msgstr ""
msgid "GroupSAML|Active SAML Group Links (%{count})" msgid "GroupSAML|Active SAML Group Links (%{count})"
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment