Skip to content
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

Acti indicator query playbook #18733

Conversation

satyakidroid
Copy link
Contributor

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

  • In Progress
  • Ready
  • In Hold - (Reason for hold)

Related Issues

fixes: link to the issue

Description

This PR is created to continue working on (#18725)

Screenshots

Paste here any images that will help the reviewer

Minimum version of Cortex XSOAR

  • 6.0.0
  • 6.1.0
  • 6.2.0
  • 6.5.0

Does it break backward compatibility?

  • Yes
    • Further details:
  • No

Must have

  • Tests
  • Documentation

@kgal-pan kgal-pan self-assigned this Apr 25, 2022
@satyakidroid
Copy link
Contributor Author

@kgal-pan I think this should work

@kgal-pan
Copy link
Contributor

@satyakidroid - Looks good. Please merge from master to fix validation:

/Utils/git_pull_master_into_fork.sh

@satyakidroid
Copy link
Contributor Author

@kgal-pan , I think the current state is good.

@kgal-pan
Copy link
Contributor

kgal-pan commented Apr 27, 2022

@yaakovi - This PR was already reviewed by @tomer-pan and approved in #18401 regarding Playbooks.

The only difference between the closed internal PR and this one is that this one includes some fixes for UTs that were sending out HTTP requests to Accenture APIs instead of mocking them, see discussion here.

Therefore, not sure that we need @idovandijk to review this PR.

@yaakovi yaakovi requested review from tomer-pan and removed request for idovandijk May 1, 2022 04:12
@yaakovi yaakovi merged commit e109963 into demisto:contrib/iDefense_ACTIIndicatorQuery_playbook May 1, 2022
yaakovi pushed a commit that referenced this pull request May 1, 2022
* Layout identification bug fixed

* Modified layout for alerts and reports

* Added playbooks

* Updated README

* added user agent

* minor changes

* minor changes

* minor changes

* Added playbooks and modified README

* Added description to Indicator Fields and updated ACTI IntelGraph version

* Update reputation-ACTI_Intelligence_Alert.json

* Update reputation-ACTI_Intelligence_Alert.json

* Adding description to the indicator type.

* missing comma

* Update reputation-ACTI_Intelligence_Report.json

* Added description to playbooks

* description added in playbooks

* removed unused tasks from playbook

* Modified the way threat types were printed in report

* Minor changes

* Added README to playbooks

* Added description for layouts

* Changes made to incident enrichment playbook

* Update README.md

* Update README.md

* Update README.md

* Update playbook-ACTI_Block_Indicators_from_Incident_README.md

* Update playbook-ACTI_Block_Indicators_severity_greater_or_equal_5_README.md

* Update playbook-ACTI_Create_Relationships_README.md

* Update playbook-ACTI_IA_IR_Enrichment_README.md

* Update playbook-ACTI_Incident_Enrichment_README.md

* Update playbook-ACTI_Indicator_Enrichment_README.md

* Update playbook-ACTI_Vulnerability_Enrichment_README.md

* Update Playbook Name

* Update Playbook title

* Update playbook-ACTI_Create_Relationships.yml

* Update Playbook Name

* Description Updated

* Updating Playbook Description

* Update Playbook Description

* Updated Playbook Description.

* Updated playbook config files

* Minor Changes

* Adding markdown postprocessing logic to download inline image links include them as encoded images

* Fixing errors

* Errors Fixed : Try 2

* Fixing ACTI Feed coverage issue

* Fixing UT coverage for ACTIIndicatorQuery

* Fixing UTs and flake8 errors

* Added UT for addBaseUrlToPartialPaths func

* UT added for domain and IP not found

* UT for markdown postprocessing

* minor chnges

* resolving flake8 errors

* threatintelreport command & some minor bug fixed

* Error fix : Try 1

* Minor changes

* Fixing Nonetype object issue

* Fixed bug which was not creating relationship

* Removed Report Enrichment step in Incident Enrichment playbook

* Added user choice to block or not block indicators

* Have added DeleteContext for user choice key and added ACTI Indicator Enrichment as sub-playbook

* Added case when IA IR is not present, also have tweaked the playbook to work 1 or more reports

* Fixing Errors

* Update playbook-ACTI_Block_High_Severity_Indicators_README.md

* Update playbook-ACTI_Block_Indicators_from_an_Incident_README.md

* Update playbook-ACTI_Create_Report-Indicator_Associations_README.md

* Update playbook-ACTI_Incident_Enrichment_README.md

* Fixed bug which wasn't able to create relationship when there is 1 indicator and 1 report

* Update playbook-ACTI_Incident_Enrichment_README.md

* Update playbook-ACTI_Block_Indicators_from_an_Incident.yml

* Update playbook-ACTI_Block_High_Severity_Indicators.yml

* Update playbook-ACTI_Vulnerability_Enrichment.yml

* Update playbook-ACTI_Vulnerability_Enrichment_README.md

* Update playbook-ACTI_Block_Indicators_from_an_Incident_README.md

* Update playbook-ACTI_Block_High_Severity_Indicators_README.md

* Update playbook-ACTI_Create_Report-Indicator_Associations.yml

* Update playbook-ACTI_Create_Report-Indicator_Associations_README.md

* Update playbook-ACTI_Create_Report-Indicator_Associations_README.md

* Update playbook-ACTI_Report_Enrichment.yml

* Update playbook-ACTI_Report_Enrichment_README.md

* Update playbook-ACTI_Indicator_Enrichment.yml

* Update playbook-ACTI_Indicator_Enrichment_README.md

* Minor changes

* Update README.md

* Fixing UT

* Added UT for enocded_images

* Resolved flake8 errors

* Made few changes to add test.com instead of intelgraph.idefense.com to mocker

Co-authored-by: Ankit Mordhwaj <[email protected]>
Co-authored-by: nirmalneupane <[email protected]>
Co-authored-by: Nirmal Neupane <[email protected]>

Co-authored-by: Satyaki Chakraborti <[email protected]>
Co-authored-by: Ankit Mordhwaj <[email protected]>
Co-authored-by: nirmalneupane <[email protected]>
Co-authored-by: Nirmal Neupane <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Contribution Form Filled Whether contribution form filled or not. Contribution Thank you! Contributions are always welcome! docs-approved Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants