From 2067e7171b14a0b3cc0dbe390c8b34b571f6ba7e Mon Sep 17 00:00:00 2001 From: thiago Date: Thu, 27 Nov 2014 14:08:34 -0200 Subject: [PATCH] Remove log write_file and identation --- lib/omniauth/strategies/remote_user.rb | 92 +++++++++++++++++++++++++++++--------------------------------------------------------------- 1 file changed, 29 insertions(+), 63 deletions(-) diff --git a/lib/omniauth/strategies/remote_user.rb b/lib/omniauth/strategies/remote_user.rb index e0900d9..f4166df 100644 --- a/lib/omniauth/strategies/remote_user.rb +++ b/lib/omniauth/strategies/remote_user.rb @@ -4,78 +4,48 @@ module OmniAuth include OmniAuth::Strategy - #option :cookie, 'rack.session' + option :cookie, 'rack.session' option :cookie, '_gitlab_session' option :internal_cookie, '_remote_user' - - def __write_file message - file = File.open("/home/git/gitlab/log/remote_user.log",'a') - file.write " \n #{message} \n" - file.close - end - - def call(env) - __write_file "Call \n" - remote_user = env['HTTP_REMOTE_USER'] - - __write_file " ... Aqui esta o remote user #{remote_user}\n" - - session_user = __current_user(env) - __write_file " .....Aqui esta o session user == #{session_user}\n" - - if ! is_in_logout? (env) - if remote_user - if session_user - if remote_user == session_user - __write_file "Entrei no remote_user == session_user" - super(env) - else - __write_file "Entrei no remote_uer != session user com session user " - __logout(env) - end - - else - __write_file "Estou sem session+_user= #{session_user}" - __login(env, remote_user) - end - - else - - if session_user - __write_file "Estou sem remote user e com session user = #{session_user}" - __logout(env) - else - __write_file "Estou sem remote user e sem session user = #{session_user}" - super(env) - end - end - else - super env - end - + if ! is_in_logout? (env) + if remote_user + if session_user + if remote_user == session_user + super(env) + else + __logout(env) + end + else + __login(env, remote_user) + end + else + if session_user + __logout(env) + else + super(env) + end + end + else + super env + end end - def is_in_logout? (env) request = Rack::Request.new(env) - __write_file "REQUEST PATH = #{request.path}" - request.path == '/users/sign_out' + request.path == '/users/sign_out' end def __current_user(env) - __write_file "__CURRENT_USER" request = Rack::Request.new(env) - __write_file "REQUEST PATH = #{request.path}" request.cookies.has_key?(options.internal_cookie) && request.cookies[options.internal_cookie] end def __logout(env) - __write_file "__LOGOUT" request = Rack::Request.new(env) response = redirect_if_not_logging_in(request, request.path) if response @@ -86,8 +56,7 @@ module OmniAuth end end - def __login(env, uid) - __write_file "__LOGIN" + def __login(env, uid) request = Rack::Request.new(env) response = redirect_if_not_logging_in(request, '/users/auth/RemoteUser') if response @@ -97,11 +66,11 @@ module OmniAuth end def redirect_if_not_logging_in(request, url) - + if ! [ - '/users/auth/RemoteUser', - '/users/auth/RemoteUser/callback' - ].include?(request.path_info) + '/users/auth/RemoteUser', + '/users/auth/RemoteUser/callback' + ].include?(request.path_info) response = Rack::Response.new response.redirect url response @@ -114,7 +83,6 @@ module OmniAuth info do user_data = request.env['HTTP_REMOTE_USER_DATA'] - __write_file "#{user_data} \n" if user_data data = JSON.parse(user_data) data['nickname'] = data['name'] @@ -125,10 +93,8 @@ module OmniAuth end def request_phase - __write_file "request phase\n" - redirect "/users/auth/RemoteUser/callback" + redirect "/users/auth/RemoteUser/callback" end end end end - -- libgit2 0.21.2