[Bug]: API /comment/list with max_depth ignores limit

[Demigodrick]

Requirements

• Is this a bug report? For questions or discussions use https://lemmy.ml/c/lemmy_support
• Did you check to see if this issue already exists?
• Is this only a single bug? Do not put multiple bugs in one issue.
• Is this a backend issue? Use the lemmy-ui repo for UI / frontend issues.

Summary

The API endpoint /comment/list has a modifier for limit so you can limit how many comments it returns.

When setting max_depth to any value (i.e 1) the API returns 300 comments, ignoring the limit.

Steps to Reproduce

1. Call /comment/list with max depth set to a value

Technical Details

Output from running code:

2024-02-02 10:20:18,122 - INFO - Requesting API Request.GET /comment/list
Requesting 20 latest comments without max depth: 20
2024-02-02 10:20:18,306 - INFO - Requesting API Request.GET /comment/list
Requesting 20 latest comments with max depth set to 1: 300

Version

0.19.3

Lemmy Instance URL

lemmy.zip

[Nutomic]

Duplicate of #3065