Skip to content

Conversation

@julienloizelet
Copy link
Collaborator

@julienloizelet julienloizelet commented Dec 12, 2025

This is the continuation of #21

To simplify maintenance, it just uses a "Watcher" client (and not 2 separate clients "Watcher" and "Alerts").

This Watcher client aims to be able to handle "Operations about watchers" shown here: https://crowdsecurity.github.io/api_doc/index.html?urls.primaryName=LAPI#/watchers

This PR enables watcher some operations (alert push/search/delete/get). It refactors common logic into AbstractLapiClient and introduces comprehensive configuration/payload validation.

@julienloizelet julienloizelet self-assigned this Dec 12, 2025
@julienloizelet
Copy link
Collaborator Author

Hi @gam6itko,

I've created this new PR from your work.

May I ask you to verify whether this implementation would meet your needs?

I mainly refactored the code so that it only uses a new "Watcher" client. (I felt it was simpler and consistent with the existing "Bouncer" client)

Thanks

@gam6itko
Copy link
Contributor

@julienloizelet LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants