Page MenuHomePhabricator

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

Description

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

Details

Reference
bz63799

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

https://gerrit.wikimedia.org/r/125364

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

https://gerrit.wikimedia.org/r/125364