Commit e1d350f8 authored by Dmitriy Zaporozhets's avatar Dmitriy Zaporozhets

Merge branch 'jej/utils-override-from-strategies-group-saml' into 'master'

Use Utils::Override from Strategies::GroupSaml

See merge request gitlab-org/gitlab-ee!8718
parents 1eaa548c cf9b7897
module OmniAuth
module Strategies
class GroupSaml < SAML
extend ::Gitlab::Utils::Override
option :name, 'group_saml'
option :callback_path, ->(env) { callback?(env) }
override :setup_phase
def setup_phase
if metadata_phase?
require_discovery_token
......@@ -22,6 +25,7 @@ module OmniAuth
# Prevent access to SLO endpoints. These make less sense at
# group level and would need additional work to securely support
override :other_phase
def other_phase
if metadata_phase?
super
......
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