Page MenuHomePhabricator

CirrusSearch should have some upper limit on the "offset" parameter
Closed, ResolvedPublic


It possible to set the offset parameter so high that it hurts the Elasticsearch. lsearchd's limit was a million and that was on that Java side. Cirrus will have to add it on the php side.

Version: unspecified
Severity: normal



Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 3:14 AM
bzimport added a project: CirrusSearch.
bzimport set Reference to bz63799.

Change 125364 had a related patch set uploaded by Manybubbles:
Add a max offset parameter

Change 125364 merged by jenkins-bot:
Add a max offset parameter