about summary refs log tree commit diff
diff options
context:
space:
mode:
authorzunda <zundan@gmail.com>2022-02-18 19:21:21 +0000
committerGitHub <noreply@github.com>2022-02-18 20:21:21 +0100
commitf9e7f2e4091cdeebfbdea47a49f020504dba038c (patch)
treef7b7dfb8b5a863b388bfc20844790163731a830b
parent1de2e3f980549ce48eeeef99b99ac64987fcce44 (diff)
Avoid return within block (#17590)
This prevents the error: LocalJumpError (unexpected return)
-rw-r--r--config/initializers/rack_attack.rb10
1 files changed, 7 insertions, 3 deletions
diff --git a/config/initializers/rack_attack.rb b/config/initializers/rack_attack.rb
index 80c005f81..a1fea7712 100644
--- a/config/initializers/rack_attack.rb
+++ b/config/initializers/rack_attack.rb
@@ -82,9 +82,13 @@ class Rack::Attack
   end
 
   throttle('throttle_sign_up_attempts/ip', limit: 25, period: 5.minutes) do |req|
-    return unless req.post? && req.path == '/auth'
-    return req.remote_ip.mask(64) if req.remote_ip.ipv6?
-    req.remote_ip
+    if req.post? && req.path == '/auth'
+      if req.remote_ip.ipv6?
+        req.remote_ip.mask(64)
+      else
+        req.remote_ip
+      end
+    end
   end
 
   throttle('throttle_password_resets/ip', limit: 25, period: 5.minutes) do |req|