Make sure we return after fail!

gitclear · Aug 26, 2020 · 6d291b2 · 6d291b2
1 parent 55f97bb
commit 6d291b2
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/lib/omniauth/strategies/azure.rb b/lib/omniauth/strategies/azure.rb
@@ -85,7 +85,9 @@ def request_phase
       # credentials at the authorization endpoint.
       def callback_phase
         error = request.params['error_reason'] || request.params['error']
-        fail!(error) if error
+        if error
+          fail!(error) and return
+        end
         @session_state = request.params['session_state']
         @id_token = request.params['id_token']
         @code = request.params['code']