Repository navigation
Notify of failed logins only for existing accounts (#172) - #238
Conversation
New setting notifyFailedOnlyExisting: failed login notifications are only sent if the entered username or email address belongs to an existing account. Other attempts are still recorded and counted. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013HDvAhe4W1FxjobS8Etch2
|
The only red check is Manifest, language and package checks, and it is not caused by this PR. It fails in the I ran the same checks locally on this branch: No fix exists yet. A small change in Generated by Claude Code |
Implements #172.
The failed login notification mail is only sent if the new setting
notifyFailedOnlyExistingis off, or the username (or email address, as suggested in the issue comments) that was entered belongs to an existing account. The check uses the name as typed, since the stored entry only holds a hash for unknown names. Other attempts are still recorded and counted towards blocking.The setting itself (default: disabled, so existing behaviour is unchanged) is added to the component's settings form in the companion PR in com_bfstop. Includes en-GB and de-DE strings and a changelog entry.
Not tested end to end: I did not run the integration test suite (needs a Joomla database); the PHP lints cleanly.
🤖 Generated with Claude Code
https://claude.ai/code/session_013HDvAhe4W1FxjobS8Etch2
Generated by Claude Code