Page MenuHomePhabricator

Disable rate limiting by default in general
ClosedPublic

Authored by epriestley on Apr 9 2014, 6:46 PM.
Tags
None
Referenced Files
F19236689: D8736.id20718.diff
Sun, Dec 21, 7:25 PM
F19232798: D8736.id20719.diff
Sun, Dec 21, 5:28 PM
F19025431: D8736.id.diff
Nov 24 2025, 7:22 AM
F19016092: D8736.diff
Nov 23 2025, 2:52 AM
F18842428: D8736.diff
Oct 28 2025, 3:42 PM
F18833884: D8736.id.diff
Oct 26 2025, 6:27 AM
F18783508: D8736.diff
Oct 13 2025, 7:21 AM
F18746976: D8736.id.diff
Oct 3 2025, 2:06 PM
Subscribers

Details

Summary

This is still too rough to enable by default. We can turn it on for secure.phabricator.com and tweak it a bit first, and then release it in general with higher defaults and more sensible behavior in edge cases.

Test Plan

Loaded some pages.

Diff Detail

Repository
rP Phabricator
Branch
noratelimit
Lint
Lint Errors
SeverityLocationCodeMessage
Errorsupport/PhabricatorStartup.php:317XHP31Use Of PHP 5.3 Features
Advicesupport/PhabricatorStartup.php:45XHP16TODO Comment
Unit
No Test Coverage

Event Timeline

epriestley retitled this revision from to Disable rate limiting by default in general.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: sophiebits, btrahan.
sophiebits edited edge metadata.
This revision is now accepted and ready to land.Apr 9 2014, 6:48 PM
epriestley updated this revision to Diff 20719.

Closed by commit rPc887871d30b7 (authored by @epriestley).