P3-05: /precedents endpoint returns ALL precedents unbounded — no pagination #23

New issue

Closed

opened 2026-06-16 13:57:02 +00:00 by Artur · 0 comments

Artur commented

2026-06-16 13:57:02 +00:00

Owner

Severity: P3 (Low)
File: decider/server.py line 189

Problem

elif self.path == "/precedents":
    ...
    results = _decider_instance.prec.search("")
    self._json(200, {"count": len(results), "precedents": results})

prec.search("") returns ALL precedents with no limit. With hundreds of precedents, this payload grows unbounded.

Fix

Add query parameters for pagination:

?limit=20&offset=0 (or ?page=1&per_page=20)
Default limit of 50
Include total count and next/prev links

**Severity**: P3 (Low) **File**: `decider/server.py` line 189 ## Problem ```python elif self.path == "/precedents": ... results = _decider_instance.prec.search("") self._json(200, {"count": len(results), "precedents": results}) ``` `prec.search("")` returns ALL precedents with no limit. With hundreds of precedents, this payload grows unbounded. ## Fix Add query parameters for pagination: - `?limit=20&offset=0` (or `?page=1&per_page=20`) - Default limit of 50 - Include total count and next/prev links

Artur added the

enhancement

reliability

P3-Low

labels

2026-06-16 13:57:02 +00:00

Artur referenced this issue from a commit

2026-06-16 13:59:15 +00:00

fix: P2-01, P2-04, P3-01, P3-02, P3-03, P3-04, P3-05

Artur closed this issue

2026-06-16 14:24:07 +00:00