[openstreetmap/openstreetmap-website] Implement allow list for account creation (PR #4407)

Branko Kokanovic notifications at github.com
Fri Dec 15 08:43:05 UTC 2023


@stalker314314 commented on this pull request.



> @@ -355,6 +355,8 @@ def check_signup_allowed(email = nil)
                    domain_mx_servers(domain)
                  end
 
+    return true if Acl.allow_account_creation(request.remote_ip, :domain => domain, :mx => mx_servers)

I think blocking should have higher priorty (to be evaluated first) than allowing

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4407#pullrequestreview-1783367266
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/4407/review/1783367266 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20231215/e9d874a8/attachment.htm>


More information about the rails-dev mailing list