Page MenuHomePhabricator

Rate limit multi-factor actions
ClosedPublic

Authored by epriestley on Apr 30 2014, 4:21 PM.
Tags
None
Referenced Files
F19940511: D8911.id21146.diff
Wed, Apr 15, 2:38 AM
F19914587: D8911.id21167.diff
Fri, Apr 3, 1:00 PM
F19891090: D8911.id.diff
Thu, Mar 26, 11:23 PM
F19848311: D8911.id21146.diff
Mar 12 2026, 7:00 PM
F19840937: D8911.id21146.diff
Mar 11 2026, 8:03 PM
F19840930: D8911.id21146.diff
Mar 11 2026, 8:03 PM
F19668293: D8911.diff
Feb 7 2026, 6:36 PM
F19524295: D8911.diff
Jan 17 2026, 11:38 PM
Subscribers

Details

Reviewers
btrahan
Maniphest Tasks
T4398: Implement two-factor authentication
Commits
Restricted Diffusion Commit
rP23e654ec2bc8: Rate limit multi-factor actions
Summary

Ref T4398. Prevent users from brute forcing multi-factor auth by rate limiting attempts. This slightly refines the rate limiting to allow callers to check for a rate limit without adding points, and gives users credit for successfully completing an auth workflow.

Test Plan

Tried to enter hisec with bad credentials 11 times in a row, got rate limited.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Rate limit multi-factor actions.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Apr 30 2014, 6:32 PM
epriestley updated this revision to Diff 21167.

Closed by commit rP23e654ec2bc8 (authored by @epriestley).