add new cloudflare ruleset commands#44533
Conversation
|
Thank you for your contribution. Your generosity and caring are unrivaled! Make sure to register your contribution by filling the Contribution Registration form, so our content wizard @kamalq97 will know the proposed changes are ready to be reviewed. |
|
Hi @andrew-paloalto, thanks for contributing to the XSOAR marketplace. To receive credit for your generous contribution please follow this link. |
🤖 AI-Powered Code Review AvailableHi @kamalq97, you can leverage AI-powered code review to assist with this PR! Available Commands:
|
|
🤖 Analysis started. Please wait for results... |
🤖 AI Review DisclaimerThis review was generated by an AI-powered tool and may contain inaccuracies. Please be advised, and we extend our sincere apologies for any inconvenience this may cause. |
marketplace-ai-reviewer
left a comment
There was a problem hiding this comment.
Hi! Thanks for your contribution adding new ruleset commands to the Cloudflare WAF integration. I've left a few notes to help polish the PR, mostly focusing on standardizing descriptions and improving code robustness.
Please ensure you use .get() when accessing command arguments, add error handling for JSON parsing, and standardize the YAML descriptions (starting with "The" and ending with a period). Also, remember to update the README with the new commands and adjust the release notes version bump to reflect the new features rather than just a patch.
Thanks for your hard work!
Additionally, please address the following file-level notes:
Packs/CloudflareWAF/Integrations/CloudflareWAF/README.md: The new ruleset management commands are missing from the README.md.Packs/CloudflareWAF/Integrations/CloudflareWAF/CloudflareWAF.yml: The first key defined at the root level of the.ymlfile must always benameordisplay. Currently, the file starts withsectionorder. Please reorder the keys so thatnameordisplayis at the top.
Additionally, please ensure that the README.md file is updated to include documentation for the newly added commands (cloudflare-waf-ruleset-list, cloudflare-waf-ruleset-get, cloudflare-waf-ruleset-create, cloudflare-waf-ruleset-update, cloudflare-waf-ruleset-delete).
Packs/CloudflareWAF/pack_metadata.json: - The mandatory keycreatedis missing from the pack metadata.- The vendor name 'Cloudflare' is missing from the
keywordsarray. Packs/CloudflareWAF/ReleaseNotes/1_0_38.md: The version bump type (patch) does not match the nature of the changes (new features).
@TheL0L please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.
TheL0L
left a comment
There was a problem hiding this comment.
Thank you @andrew-paloalto for your contribution!
Aside from refactoring the two for-loops mentioned in the comments above, everything looks great.
Please address the remaining feedback from @marketplace-ai-reviewer so we can get this merged.
|
Hi @andrew-paloalto! Just a quick heads-up that this PR will be marked as stale and closed next week. If you still want to push this through, please tackle the remaining feedback from @marketplace-ai-reviewer. Let me know if you need any help! |
|
Hi @andrew-paloalto, please address the failing unit test and execute the pre-commit hooks locally. Once the build passes, we can proceed with scheduling a demo. |
|
For the Reviewer: Trigger build request has been accepted for this contribution PR. |
|
For the Reviewer: Successfully created a pipeline in GitLab with url: https://gitlab.xdr.pan.local/xdr/cortex-content/content/-/pipelines/10424594 |
|
For the Reviewer: Trigger build request has been accepted for this contribution PR. |
|
For the Reviewer: Successfully created a pipeline in GitLab with url: https://gitlab.xdr.pan.local/xdr/cortex-content/content/-/pipelines/10643509 |
|
Validate summary Verdict: PR can be force merged from validate perspective? ❌ |
4d43a26
into
demisto:contrib/andrew-paloalto_cloudflare-ruleset-list
|
Thank you for your contribution. Your external PR has been merged and the changes are now included in one or more internal PRs for further review. Note Contributions containing XSIAM content (Modeling Rules, Parsing Rules, Correlation Rules, or Dashboards) are subject to an additional review process and may take longer to be merged to master. |
* add new cloudflare ruleset commands (#44533) * add new cloudflare ruleset commands * address failing tests * ai reviewer feedback * additional requested changes * AI reviewed comments * unit test fix * ruff * md linting * adding additional param checks to avoid user errors * update docker image --------- Co-authored-by: Andrew Scott <77340714+andrew-paloalto@users.noreply.github.com> Co-authored-by: Sergei Yakima <syakima@paloaltonetworks.com>
Contributing to Cortex XSOAR Content
Make sure to register your contribution by filling the contribution registration form
The Pull Request will be reviewed only after the contribution registration form is filled.
Status
Related Issues