gethomepage / homepage

A highly customizable homepage (or startpage / application dashboard) with Docker and service API integrations.
https://gethomepage.dev
GNU General Public License v3.0
18.73k stars 1.1k forks source link

Fix: query crowdsec api without limit #3790

Closed Cyb3rzombie closed 2 months ago

Cyb3rzombie commented 2 months ago

Proposed change

The crowdsec widget queries the Crowdsec LAPI to get the amount of alerts and bans. Apparently there is a default limit of 100.

grafik

Setting the limit parameter to 0 removes that limit and shows the correct number of alerts/bans:

grafik

Link to the Crowdsec LAPI documentation

Closes # (no issue)

Type of change

Checklist:

shamoon commented 2 months ago

The crowdsec api is sorta crazy and returns massive amounts of data so removing this limit is unwise. Many users currently complain about the warnings for large API calls (>4mb). Someone ought to PR the crowdsec api for improvements.

github-actions[bot] commented 1 month ago

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion for related concerns. See our contributing guidelines for more details.