Rate-limit all emails sent, fix GHCR tags issue, and change "Smartphone" to "Authenticator"
What's Changed
- don't push branch tags to GHCR because it doesn't like tags with a slash by @briskt in #385
- [IDP-1266, part 1] Track emails to non-users by @forevermatt in #384
- Release - replace Smartphone with Authenticator by @hobbitronics in #386
- [IDP-1266, part 2] Rate limit both non user emails, add EmailLog validation by @forevermatt in #387
Changelog
Fixed
- Don't push branch tags to GHCR because it doesn't like tags with a slash
- Add 'ext-group-sync-errors' and 'abandoned-users' EmailLog message types
- Fix argument within
iCheckIfAGetBackupCodesEmailHasBeenSentRecently()
- It was passing a
User
when it should have been passing the User ID.
- It was passing a
- Use reflection to avoid need to maintain duplicate list of message types
- Add a test that the 'abandoned-user' email isn't sent too frequently
- Add a test that the 'ext-groups-sync-errors' email isn't sent too frequently
- Remove unneeded
use
import in a test file - Ensure we don't log an email as to both a User and non-user address
- Ensure every email we log is to either a User or a non-user address
Changed (non-breaking)
- Allow
email_log.user_id
to be null, and addnon_user_address
column - Renamed "Smartphone" to "Authenticator" (re: TOTP MFA)
- Avoid sending 'ext-groups-sync-errors' email too frequently
- Log emails sent to non-user addresses, too
Full Changelog: 6.10.4...6.10.5