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

Update RSANetWitnessv115.py with fetch alerts api call #37261

Open
wants to merge 1 commit into
base: contrib/adit-ghildiyal_master
Choose a base branch
from

Conversation

adit-ghildiyal
Copy link
Contributor

Update RSANetWitnessv115.py with fetch alerts api call

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

A few sentences describing the overall goals of the pull request's commits.

Must have

  • Tests
  • Documentation

Update RSANetWitnessv115.py with fetch alerts api call
@content-bot content-bot added Contribution Thank you! Contributions are always welcome! External PR Partner Support Level Indicates that the contribution is for Partner supported pack labels Nov 18, 2024
@content-bot content-bot changed the base branch from master to contrib/adit-ghildiyal_master November 18, 2024 11:54
@content-bot
Copy link
Collaborator

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 @amshamah419 will know the proposed changes are ready to be reviewed.
For your convenience, here is a link to the contributions SLAs document.

@content-bot
Copy link
Collaborator

Hi @adit-ghildiyal, thanks for contributing to the XSOAR marketplace. To receive credit for your generous contribution please follow this link.

})

response = self._http_request(
'GET', f'rest/api/alert/fetch', data=data
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
'GET', f'rest/api/alert/fetch', data=data
'GET', 'rest/api/alert/fetch', data=data

Copy link
Contributor

@amshamah419 amshamah419 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @adit-ghildiyal thanks for your contribution! Do you have some documentation about the API endpoint change?

Also, we should bump the pack version when making changes. You can do this via the demisto-sdk by running demisto-sdk update-release-notes -i Packs/RSANetWitness_v11_1

@@ -91,10 +91,23 @@ def incident_add_journal_entry_request(self, id_: str | None, author, notes: str

def incident_list_alerts_request(self, page_size: str | None, page_number: str | None, id_: str | None) -> dict:
params = assign_params(pageNumber=page_number, pageSize=page_size)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did they change the API endpoint here? I ask because it looks like we are no longer using the page_number and page_size arguments. If the new endpoint supports it, we should try and keep this functionality.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Contribution Thank you! Contributions are always welcome! External PR Partner Support Level Indicates that the contribution is for Partner supported pack
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants