Add registration rate limit

[ArtOfCode-]

Adds a configurable rate limit for how quickly accounts can be created from the same IP. Related to the ongoing email issues.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.36%. Comparing base (51cbe3e) to head (64846ef).
Report is 12 commits behind head on develop.

Additional details and impacted files

Components	Coverage Δ
controllers	61.81% <100.00%> (+0.15%)	⬆️
helpers	69.17% <ø> (ø)
jobs	28.00% <ø> (ø)
models	81.92% <ø> (-0.02%)	⬇️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Oaphi]

You cannot call create unless the parent is saved

Unfortunately, send_welcome_tour_message causes this error as it was supposed to be an after_create callback. The same problem is likely to apply to the ensure_websites callback

[ArtOfCode-]

Where are you seeing that? I haven't hit that error anywhere

[Oaphi]

Where are you seeing that? I haven't hit that error anywhere

Any attempt to manually (via the UI) sign up a user with the PR checked out - do you not get this error, @ArtOfCode-?

[ArtOfCode-]

Nope, no error in manual testing or the automated tests

[Oaphi]

Nope, no error in manual testing or the automated tests

Hmm... Tests pass for me, but not the manual registration - not sure what's going on.

Nevermind, I think I know what's going on: there were validation errors that prevented the user from being saved. save in Devise::RegistrationsController fails, but as its return value is not handled, the user is passed back in the "dirty" state. Let's check whether the user has any errors before even trying to check the limit?

[Oaphi]

LGTM