Paths

Table of Contentst

Disable rate limiting by default in general
ClosedPublic
Actions

Authored by epriestley on Apr 9 2014, 6:46 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Jan 22, 5:08 AM

	Unknown Object (File)
	Wed, Jan 22, 5:08 AM

	Unknown Object (File)
	Wed, Jan 22, 4:22 AM

	Unknown Object (File)
	Tue, Jan 21, 3:01 PM

	Unknown Object (File)
	Tue, Jan 21, 9:09 AM

	Unknown Object (File)
	Fri, Jan 17, 3:51 PM

	Unknown Object (File)
	Fri, Jan 17, 6:02 AM

	Unknown Object (File)
	Dec 19 2024, 4:46 PM

View All Files

Subscribers

epriestley

Details

Reviewers

btrahan
sophiebits

Commits

Restricted Diffusion Commit
rPc887871d30b7: Disable rate limiting by default in general

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

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 20718.Apr 9 2014, 6:46 PM

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.

Herald added a subscriber: epriestley.Apr 9 2014, 6:46 PM

sophiebits accepted this revision.Apr 9 2014, 6:48 PM

sophiebits edited edge metadata.

This revision is now accepted and ready to land.Apr 9 2014, 6:48 PM

Closed by commit rPc887871d30b7 (authored by @epriestley).

epriestley mentioned this in T12612: Design (and possibly implement) production rate limiting controls.Apr 21 2017, 2:02 PM

Revision Contents
Changeset List

Path

Size

support/

PhabricatorStartup.php

6 lines

Diff 20719

View Options

support/PhabricatorStartup.php

Disable rate limiting by default in generalClosedPublicActions