Avoid return within block (#17590)

This prevents the error: LocalJumpError (unexpected return)

diff --git a/config/initializers/rack_attack.rb b/config/initializers/rack_attack.rb
index 80c005f81983994cbf0fad7beca7a122b4d741aa..a1fea7712de7cb9ccc29d70ae3117eacfc5925f0 100644 (file)
--- 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|