Push review · 1379c9c
Add active-user listing to UserService
GitHub check
All clear
No verified vulnerabilities and code craft 300/300.
Points
0ptsTotal10001000
Security700700
Craft300300
What changed: Adds a paginated, parameterized active-user query with a hard page-size cap.Scripted demo replay
Security agent
Scripted demo replay
Instead of reading one big prompt, the agent plans, then investigates the code graph one tool call at a time and sees each real result before deciding what to check next.
Plan
- 1Inspect the changed function and where its inputs come from
- 2Trace entry points through the change to dangerous calls
- 3Read the code on the path for validation or auth checks
- 4Report only what the evidence supports
- Step 1Looked up
app.services.user_service:UserService.list_activeStart with the changed symbol.
What it found
{ "id": "app.services.user_service:UserService.list_active", "kind": "method", "file": "app/services/user_service.py", "lines": [ 13, 20 ], "params": [ "self", "limit" ], "entry_point": false, "route": null, "depends_on": [], "untrusted_inputs": [], "dangerous_calls": [], "has_docstring": true } callers: (none) callees: app.db:Database.fetch_all (app/db.py:20-24) [sinks=sql@L23(tainted)] via line 15 [tainted args]