Aias00 opened a new issue, #6673:
URL: https://github.com/apache/shenyu/issues/6673

   - severity: High; files: `ApiServiceImpl.java:261-273`; SQL 
`mappers/api-sqlmap.xml:268-301`
   - description: (1) `ApiServiceImpl` has no `@Pageable` (verified 0 
occurrences); `PageResultUtils.result(pageParameter, listSupplier)` (2-arg 
form, `PageResultUtils.java:55-57`) applies no SQL LIMIT — full result set 
loaded. (2) Per-row N+1: `tagRelationMapper.selectByQuery(apiId)` then 
`tagMapper.selectByIds(tagIds)` per API; `tag_relation` has no index on 
`api_id`/`tag_id`. (3) List selects `api.document` (LONGVARCHAR/TEXT) for every 
row though a list view does not need it.
   - impact: Unbounded memory + row count × 2 round-trips; can OOM/stall on 
large inventories.
   - suggested_fix: Add `@Pageable`; exclude `document` from the list column 
list; batch-fetch tag relations for all page ids in one query.
   - confidence: High
   
   ---
   _Identified during the 2026-08-02 deep re-scan; full list in 
[`docs/scan2-2026-08-02/00-consolidated-critical-high.md`](docs/scan2-2026-08-02/00-consolidated-critical-high.md)._


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to