Skip to content

Feature add alert query commands #384

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 11 commits into from
Aug 4, 2025

Conversation

Noble-47
Copy link
Contributor

In respect to #383

Integrated the alerts_api into buffacli commands for alert querying and introduced a new module render which determines how rich renderables are printed. There are four rendering options:

  • less
  • paginate
  • shell
  • default
  • default only prints the renderable to the screen with no form of pagination added
  • less uses Unix less as the pager for printing
  • paginate paginates the API call rather than the content to be printed out (hasn't been implemented)
  • shell creates an interactive shell for more complex pattern matching (future feature)

Depends on #335

Noble-47 added 9 commits July 2, 2025 10:25
* Restructured code base
* Added command to print out ingestion sources, configuration and active
  ingestion
* Reformated formatters function doc string
* Added Alerters model
* Added request functions to retrieve alerters
* Added command function in `show.py`
* Added tests
* Created AlertQuery model
* Defined alery query request method
* Created command function for querying alerts
* Moved printing responsiblity to render module
@Lorygold
Copy link
Collaborator

Lorygold commented Aug 1, 2025

@Noble-47 there are some conflicts

@Noble-47 Noble-47 marked this pull request as ready for review August 1, 2025 12:35
@Noble-47
Copy link
Contributor Author

Noble-47 commented Aug 1, 2025

@Noble-47 there are some conflicts

The conflicts have been resolved.

@Lorygold Lorygold requested review from ManofWax and Lorygold August 1, 2025 16:33
@Lorygold Lorygold merged commit f733a2a into certego:buffacli Aug 4, 2025
2 checks passed
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