Merge pull request #1698 from ClearlyClaire/glitch-soc/merge-upstream

Merge upstream changes
2 years ago · e0d94323bc
parent 23a548d794 2bd9bcf698
commit e0d94323bc
1 changed files with 4 additions and 5 deletions
--- a/config/initializers/rack_attack.rb
+++ b/config/initializers/rack_attack.rb
@ -83,11 +83,10 @@ class Rack::Attack

  throttle('throttle_sign_up_attempts/ip', limit: 25, period: 5.minutes) do |req|
    if req.post? && req.path == '/auth'
-      if req.remote_ip.ipv6?
-        req.remote_ip.mask(64)
-      else
-        req.remote_ip
-      end
+      addr = req.remote_ip
+      addr = IPAddr.new(addr) if addr.is_a?(String)
+      addr = addr.mask(64) if addr.ipv6?
+      addr.to_s
    end
  end