From 6d291b2f0b62a68abc7cc1ce52de6b1716005998 Mon Sep 17 00:00:00 2001 From: Matthew Kloster Date: Wed, 26 Aug 2020 14:22:40 -0700 Subject: [PATCH] Make sure we `return` after `fail!` --- lib/omniauth/strategies/azure.rb | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/lib/omniauth/strategies/azure.rb b/lib/omniauth/strategies/azure.rb index beb5765..d719b38 100644 --- 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']