Send API token expiry notification emails#8290

Merged

Turbo87 merged 22 commits intorust-lang:mainfrom Rustin170506:rustin-patch-expiry_notification_job

May 15, 2024

+240

Commits on May 15, 2024

feat: add expiry_notification_at migration

Rustin170506
authored and
Turbo87
committed
feat: add CheckAboutToExpireToken job

Rustin170506
authored and
Turbo87
committed
feat: find tokens expiring with couple days

Rustin170506
authored and
Turbo87
committed
feat: send emails

Rustin170506
authored and
Turbo87
committed
refactor: better email template

Rustin170506
authored and
Turbo87
committed
test: add an integration test

Rustin170506
authored and
Turbo87
committed
feat: add a check about to expiry command

Rustin170506
authored and
Turbo87
committed
fix: skip failed email

Rustin170506
authored and
Turbo87
committed
fix: do not send email for expired token

Rustin170506
authored and
Turbo87
committed
test: non-expired tokens are not affected

Rustin170506
authored and
Turbo87
committed
refactor: use chrono::TimeDelta and move find_tokens_expiring_within_days out

Rustin170506
authored and
Turbo87
committed
fix: do not expose expiry_notification_at to the API

Rustin170506
authored and
Turbo87
committed
feat: only query 10000 tokens

Rustin170506
authored and
Turbo87
committed
refactor: rename to SendTokenExpiryNotifications

Rustin170506
authored and
Turbo87
committed
fix: use handle_expiring_token to send email

Rustin170506
authored and
Turbo87
committed
worker/jobs/expiry_notification: Simplify filter() call
Turbo87
committed
worker/jobs/expiry_notification: Remove unnecessary parentheses
Turbo87
committed
worker/jobs/expiry_notification: Simplify now() imports
Turbo87
committed
worker/jobs/expiry_notification: Simplify diesel imports
Turbo87
committed
worker/jobs/expiry_notification: Replace TimeDelta with DateTime
Turbo87
committed
worker/jobs/expiry_notification: Adjust doc comments
Turbo87
committed
worker/jobs/expiry_notification: Replace match statement with ok_or_else()
Turbo87
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Send API token expiry notification emails#8290

Commits on May 15, 2024

feat: add expiry_notification_at migration

feat: add CheckAboutToExpireToken job

feat: find tokens expiring with couple days

feat: send emails

refactor: better email template

test: add an integration test

feat: add a check about to expiry command

fix: skip failed email

fix: do not send email for expired token

test: non-expired tokens are not affected

refactor: use chrono::TimeDelta and move find_tokens_expiring_within_days out

fix: do not expose expiry_notification_at to the API

feat: only query 10000 tokens

refactor: rename to SendTokenExpiryNotifications

fix: use handle_expiring_token to send email

worker/jobs/expiry_notification: Simplify `filter()` call

worker/jobs/expiry_notification: Remove unnecessary parentheses

worker/jobs/expiry_notification: Simplify `now()` imports

worker/jobs/expiry_notification: Simplify `diesel` imports

worker/jobs/expiry_notification: Replace `TimeDelta` with `DateTime`

worker/jobs/expiry_notification: Adjust doc comments

worker/jobs/expiry_notification: Replace `match` statement with `ok_or_else()`