Closed skrilab closed 2 weeks ago
@skrilab could we also name the namespace from integrations.checkpoint
to integrations.check_point
for consistency as well. Thanks for your suggestion for the name change!
Yes, integrations.check_point
makes sense!
[!IMPORTANT]
Review skipped
Auto reviews are limited to specific labels.
:label: Labels to auto review (1)
* coderabbitPlease check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
The changes in this pull request include updates to the checkpoint.py
file, where terminology is modified from "Checkpoint" to "Check Point," and a key is removed from the checkpoint_secret
. Additionally, the get_auth_token
function's response handling is adjusted. Two new actions, get_xdr_incidents
and update_xdr_incident
, are introduced in the integrations.check_point
namespace, allowing for the retrieval and updating of Check Point Infinity XDR incidents.
File Path | Change Summary |
---|---|
registry/tracecat_registry/integrations/checkpoint.py |
Updated terminology from "Checkpoint" to "Check Point," removed "CHECKPOINT_API_URL," and modified response handling in get_auth_token . |
registry/tracecat_registry/templates/checkpoint/get_cp_xdr_incidents.yml |
Added new action get_xdr_incidents to retrieve Check Point Infinity XDR incidents. |
registry/tracecat_registry/templates/checkpoint/update_cp_xdr_incident.yml |
Added new action update_xdr_incident to update incidents in Check Point Infinity XDR. |
Objective | Addressed | Explanation |
---|---|---|
Fix incorrect secrets API usage in get token actions (#501) |
❌ | The change does not address the specified API usage issue. |
🐇 In the meadow, bright and spry,
Check Point's name now reaches high!
With tokens fetched and incidents too,
Our actions dance, all fresh and new.
Hops of joy, let’s celebrate,
For changes made, oh, isn’t it great? 🌼
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
@skrilab please note that I've changed the namespace for the action as per our discussion above. This will cause breaking changes! You'll need to replace the actions in your "dev" environment with the new actions. Thanks!
Description
This PR fixes:
This PR removes:
Related Tickets & Documents
This is an addition to Fix #501
Screenshots/Recordings
The error:
Steps to QA
[optional] What gif best describes this PR?
:)
Summary by CodeRabbit
get_xdr_incidents
for retrieving Check Point Infinity XDR incidents from the last 7 days.update_xdr_incident
for updating incidents in the Check Point Infinity XDR system.