Skip to content

Commit e962e86

Browse files
committed
Fix elsif block
1 parent d9f3243 commit e962e86

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

lib/omniauth/strategies/oauth2.rb

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,9 +83,10 @@ def token_params
8383

8484
def callback_phase # rubocop:disable Metrics/AbcSize, Metrics/CyclomaticComplexity, Metrics/MethodLength, Metrics/PerceivedComplexity
8585
error = request.params["error_reason"] || request.params["error"]
86-
elsif !options.provider_ignores_state && (request.params["state"].to_s.empty? || !secure_compare(request.params["state"], session.delete("omniauth.state")))
8786
elsif error
8887
fail!(error, CallbackError.new(request.params["error"], request.params["error_description"] || request.params["error_reason"], request.params["error_uri"]))
88+
elsif !options.provider_ignores_state && (request.params["state"].to_s.empty? || !secure_compare(request.params["state"], session.delete("omniauth.state")))
89+
fail!(:csrf_detected, CallbackError.new(:csrf_detected, "CSRF detected"))
8990
else
9091
self.access_token = build_access_token
9192
self.access_token = access_token.refresh! if access_token.expired?

0 commit comments

Comments
 (0)