WordPress / WordPress/two-factor

Notify user/admin when sensitive events occur

Open
#476 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
PHP
Stars
825
Forks
187
Avg merge
2d 11h
Merged PRs (30d)
17

Description

Most services w/ strong security standards will notify you when security-sensitive events happen. That (hopefully) makes you aware when an attacker has compromised something.

Here are some rough thoughts on events that should trigger a notification:

User only
  • when 2fa settings changed in any way
  • when failed attempt at 2nd factor after successful password - related #462, #510
Admin and User
  • when backup code used (should be filterable off for large sites)
  • when user disables all 2fa providers (maybe not necessarily if we add a mechanism to force 2fa on for certain roles - #185, #239, #255, #307, #451 )
Admin only
  • Email provider enables automatically as a fallback (see #586)

Should anything be added to those lists?

Email notifications are good because the user doesn't have to visit the site to be informed about the event. Email is also unreliable, though, so we could consider also adding notifications in wp-admin, like #510 and #482

We might want to do #459 first, since that is similar and could lay the ground work. Or maybe we need a generic PR that just fires actions when events occur, and separate PRs for specific responses to that (email, error_log). An add-on plugin could redirect notices to Stream, monolog, sentry.io, etc).

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reviewing the proposed event list and the related issues #459, #462, #510, #586, #185, #239, #255, #307, #451 and #482. Before implementation, define whether the work should provide generic actions, email notifications, wp-admin notices, or separate responses; done requires an agreed scope and notification behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
php, wordpress
Domain
authentication, security
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.