Skip to content

Fixed get-indicators-future-attack issue#41

Open
meghak-metron wants to merge 16 commits into
masterfrom
SilentPush-indicators-fix
Open

Fixed get-indicators-future-attack issue#41
meghak-metron wants to merge 16 commits into
masterfrom
SilentPush-indicators-fix

Conversation

@meghak-metron

Copy link
Copy Markdown
Collaborator

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

Fixed get-indicators-ftuture-attack issue

Must have

  • Tests
  • Documentation

@meghak-metron meghak-metron requested a review from yash-metron May 19, 2025 07:26
@meghak-metron meghak-metron self-assigned this May 19, 2025
@yash-metron

Copy link
Copy Markdown

I have reviewed the changes, and everything appears to be the same as we discussed. Please make sure to test all the changes.

antonj-metron and others added 13 commits May 28, 2025 12:44
* Quick fix-  silentpush-get-future-attack-indicators command

* PR comments have been addressed along with validation errors. Please review when you're available.

* Fixed the PR comments and Updated SilentPush.py

* Updated Unit Tests and Release Notes

* Validation fix

* Updated Version for the App

* Updated the ReleaseNotes

* Quick Fix:-Updated ReleaseNote

* Fixed the Formatting Error
* Fixed Output context issue for nameserver reputation

* Changed output path in test.py for nameserver reputation
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.

4 participants