Open Bug 1962523 Opened 27 days ago

Lando should not set a limit when querying Phabricator

Categories

(Conduit :: Lando, enhancement)

Product:
Conduit
Component:
Lando
Type:
enhancement

Tracking

(Not tracked)

People

(Reporter: zeid, Unassigned)

Details

There are currently various methods that set a limit when calling the Phabricator API. When the limit is set to less than 100, there is no functional impact, and it appears that this is used to validate that the number of results returned match the requested items. However, since Phabricator does not return more than 100 results at a time, it raises an error when a limit that is greater than 100 is requested.

Therefore, the usage of limits does not make sense in the current implementation and should be removed altogether.

You need to log in before you can comment on or make changes to this bug.