FeedDomainTools Pack Release v1.0.7#44090
Conversation
[IDEV-2473] DomainTools Feeds: fix rawJSON for domainrisk and domainhotlist feeds
|
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. |
🤖 AI-Powered Code Review AvailableHi @kamalq97, @MosheEichler, you can leverage AI-powered code review to assist with this PR! Available Commands:
|
|
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/8615731 |
|
🤖 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 contributing the DomainTools feed integration. I've reviewed your PR and left a few notes to help polish it up. The main things to look at are adding the missing created key and vendor keyword to the pack metadata, as well as aligning a default value in the Python code with your YAML file. Thanks for your work on this!
Additionally, please address the following file-level notes:
Packs/FeedDomainTools/pack_metadata.json: - The mandatory keycreatedis missing frompack_metadata.json.- The vendor name 'DomainTools' must appear in the
keywordsarray.
@MosheEichler, @kamalq97 please review and approve the results generated by the AI Reviewer by responding 👍 on this comment.
91b0e02
into
demisto:contrib/DomainTools_domaintools-feeds-v1.0.7
|
Thank you for your contribution. Your external PR has been merged and the changes are now included in an internal PR for further review. The internal PR will be merged to the master branch within 3 business days. |
* FeedDomainTools Pack Release v1.0.7 (#44090) * fix rawJSON for domainrisk and domainhotlist feeds * update release notes * Docker * RN --------- Co-authored-by: Bri <133698148+briluza@users.noreply.github.com> Co-authored-by: Moshe Eichler <78307768+MosheEichler@users.noreply.github.com> Co-authored-by: Moshe Eichler <meichler@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
domainriskanddomainhotlistfeeds where the risk score details are not in rawJSON.Must have
relates: https://jira-dc.paloaltonetworks.com/browse/CIAC-16670