Commit 5d74ffb4 authored by Rémy Coutable's avatar Rémy Coutable

Reduce diff with CE in OmniauthCallbacksController

Signed-off-by: default avatarRémy Coutable <remy@rymai.me>
parent de62d2c4
......@@ -3,6 +3,7 @@
class OmniauthCallbacksController < Devise::OmniauthCallbacksController
include AuthenticatesWithTwoFactor
include Devise::Controllers::Rememberable
protect_from_forgery except: [:kerberos, :saml, :cas3], prepend: true
def handle_omniauth
......@@ -55,16 +56,6 @@ class OmniauthCallbacksController < Devise::OmniauthCallbacksController
handle_omniauth
end
def kerberos_spnego
# The internal kerberos_spnego provider is a replacement for
# omniauth-kerberos. Here we re-use the 'kerberos' provider name to ease
# the transition. In time (in GitLab 9.0?) we should remove the
# omniauth-kerberos gem and rename the internal 'kerberos_spnego'
# provider to plain 'kerberos' and remove this special method.
oauth['provider'] = 'kerberos'
handle_omniauth
end
def authentiq
if params['sid']
handle_service_ticket oauth['provider'], params['sid']
......
......@@ -4,6 +4,16 @@ module EE
module OmniauthCallbacksController
extend ::Gitlab::Utils::Override
def kerberos_spnego
# The internal kerberos_spnego provider is a replacement for
# omniauth-kerberos. Here we re-use the 'kerberos' provider name to ease
# the transition. In time (in GitLab 9.0?) we should remove the
# omniauth-kerberos gem and rename the internal 'kerberos_spnego'
# provider to plain 'kerberos' and remove this special method.
oauth['provider'] = 'kerberos'
handle_omniauth
end
protected
override :fail_login
......
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