WordPress / WordPress/two-factor
Notify user/admin when sensitive events occur
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
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- 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