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

[Marketplace Contribution] Dragos Worldview - Content Pack Update #30207

Conversation

xsoar-bot
Copy link
Contributor

Status

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

Contributor

@NicCas

Notes

I can send the demo video through slack

Video Link

Short demo video of the Pack usage. Speeds up the review. Optional but recommended. Use a video sharing service such as Google Drive or YouTube.

@content-bot content-bot added Contribution Thank you! Contributions are always welcome! External PR Community Support Level Indicates that the contribution is for Community supported pack labels Oct 16, 2023
@content-bot content-bot changed the base branch from master to contrib/xsoar-contrib_NicCas-contrib-DragosWorldview October 16, 2023 19:29
@content-bot
Copy link
Collaborator

Thank you for your contribution. Your generosity and caring are unrivaled! Rest assured - our content wizard @ilappe will very shortly look over your proposed changes.
For your convenience, here is a link to the contributions SLAs document.

FIxed error with missing period in description
Added a period to the end of all command, argument, and output descriptions
Added input and output tables for dragos-get-indicators()
name: dragos-get-stix2
dockerimage: demisto/python3:3.10.12.63474
dockerimage: demisto/python3:3.10.13.75921
Copy link
Contributor

Choose a reason for hiding this comment

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

@NicCas update docker image to demisto/python3:3.10.13.77674

Split username/password setup of credential into two password only ones.
Added back old encrypted parameters for backwards compatibility
Updated key and token check to support old encryption and new credential parameters.
Copy link
Contributor

@ilappe ilappe left a comment

Choose a reason for hiding this comment

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

Thanks @NicCas for your contribution!
please see my few comments
in addition, can you please,

  • provide me a command_example file as described here , we will need it to generate the README with more information about the args and context outputs.
  • prepare a short video that show the updated command run in the war room with/without the new args and see the result and the context ?
    you can send them via DFIR in slack.
    thanks again!

@ilappe ilappe added the pending-contributor The PR is pending the response of its creator label Oct 22, 2023
NicCas and others added 6 commits October 23, 2023 15:13
…view.yml

Co-authored-by: Israel Lappe <79846863+ilappe@users.noreply.github.com>
…view.yml

Co-authored-by: Israel Lappe <79846863+ilappe@users.noreply.github.com>
Added bug fix where multiple page numbers could be in an api call
Added suggested Boolean wrapper and default
@ilappe ilappe removed the pending-contributor The PR is pending the response of its creator label Oct 24, 2023
@ilappe
Copy link
Contributor

ilappe commented Oct 24, 2023

Hi @NicCas thanks for implementing the fixes!!
i see there is workflows that failed (looks like lint error).
in addition, when the PR are ready to re-review, please click on the request re-review button under the reviewers section
thanks!

@ilappe ilappe added the pending-contributor The PR is pending the response of its creator label Oct 24, 2023
@NicCas
Copy link
Contributor

NicCas commented Oct 24, 2023

Hi @NicCas thanks for implementing the fixes!! i see there is workflows that failed (looks like lint error). in addition, when the PR are ready to re-review, please click on the request re-review button under the reviewers section thanks!

I don't see this button but the changes are ready for review!

@ilappe
Copy link
Contributor

ilappe commented Oct 25, 2023

Thanks @NicCas for implementing the fixes!!
see one point that remain to check
in addition, are you able to follow the requests in this comment?

@ilappe ilappe added post-demo ready-for-instance-test In contribution PRs, this label will cause a trigger of a build with a modified pack from the PR. labels Oct 26, 2023
@content-bot
Copy link
Collaborator

content-bot commented Oct 26, 2023

For the Reviewer: Successfully created a pipeline in Gitlab with url: https://code.pan.run/xsoar/content/-/pipelines/6711212

@ilappe ilappe added ready-for-instance-test In contribution PRs, this label will cause a trigger of a build with a modified pack from the PR. and removed ready-for-instance-test In contribution PRs, this label will cause a trigger of a build with a modified pack from the PR. labels Oct 26, 2023
Copy link
Contributor

@ilappe ilappe left a comment

Choose a reason for hiding this comment

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

Thanks @NicCas for this contribution!

@ilappe ilappe merged commit 49aae38 into demisto:contrib/xsoar-contrib_NicCas-contrib-DragosWorldview Oct 29, 2023
17 of 20 checks passed
ilappe added a commit that referenced this pull request Oct 29, 2023
…0207) (#30504)

* "contribution update to pack "Dragos Worldview""

* Update DragosWorldview.yml

FIxed error with missing period in description

* Update DragosWorldview_description.md

Fixed spelling mistake

* Update DragosWorldview.yml

Added a period to the end of all command, argument, and output descriptions

* Update README.md

Added input and output tables for dragos-get-indicators()

* Changed credential for backwards compatibility

Split username/password setup of credential into two password only ones.
Added back old encrypted parameters for backwards compatibility

* Added backwards compatibility for credentials

Updated key and token check to support old encryption and new credential parameters.

* Updated docker image tag

* Added back link

* Updating docker image

* Update Packs/DragosWorldview/Integrations/DragosWorldview/DragosWorldview.yml



* Update Packs/DragosWorldview/Integrations/DragosWorldview/DragosWorldview.yml



* Update DragosWorldview.py

Added bug fix where multiple page numbers could be in an api call
Added suggested Boolean wrapper and default

* Removed redundant boolean check

* changed output's key field back to indicator_id

* Added suggested int wrapper for page

* Fixed formatting issues and updated docker

* Changed id > indicator_id in outputs

* Removed parathesis for bug fix

* Type casted raw_response[...] to int

* Added requested changes

Added descriptions to the integration parameters
Changed order of columns for !dragos-get-indicators() output table
Added example commands for all 4 integration commands

* Added human readable output for dragos-get-indicators()

* Added human readable output example for dragos-get-indicators()

* Update 1_1_3.md

* update README - fix validation

* update correct README

---------

Co-authored-by: xsoar-bot <67315154+xsoar-bot@users.noreply.github.com>
Co-authored-by: Nicole Casartelli <npcasartelli@gmail.com>
Co-authored-by: Israel Lappe <79846863+ilappe@users.noreply.github.com>
Co-authored-by: ilappe <ilappe@paloaltonetworks.com>
sapirshuker pushed a commit that referenced this pull request Dec 21, 2023
…0207) (#30504)

* "contribution update to pack "Dragos Worldview""

* Update DragosWorldview.yml

FIxed error with missing period in description

* Update DragosWorldview_description.md

Fixed spelling mistake

* Update DragosWorldview.yml

Added a period to the end of all command, argument, and output descriptions

* Update README.md

Added input and output tables for dragos-get-indicators()

* Changed credential for backwards compatibility

Split username/password setup of credential into two password only ones.
Added back old encrypted parameters for backwards compatibility

* Added backwards compatibility for credentials

Updated key and token check to support old encryption and new credential parameters.

* Updated docker image tag

* Added back link

* Updating docker image

* Update Packs/DragosWorldview/Integrations/DragosWorldview/DragosWorldview.yml



* Update Packs/DragosWorldview/Integrations/DragosWorldview/DragosWorldview.yml



* Update DragosWorldview.py

Added bug fix where multiple page numbers could be in an api call
Added suggested Boolean wrapper and default

* Removed redundant boolean check

* changed output's key field back to indicator_id

* Added suggested int wrapper for page

* Fixed formatting issues and updated docker

* Changed id > indicator_id in outputs

* Removed parathesis for bug fix

* Type casted raw_response[...] to int

* Added requested changes

Added descriptions to the integration parameters
Changed order of columns for !dragos-get-indicators() output table
Added example commands for all 4 integration commands

* Added human readable output for dragos-get-indicators()

* Added human readable output example for dragos-get-indicators()

* Update 1_1_3.md

* update README - fix validation

* update correct README

---------

Co-authored-by: xsoar-bot <67315154+xsoar-bot@users.noreply.github.com>
Co-authored-by: Nicole Casartelli <npcasartelli@gmail.com>
Co-authored-by: Israel Lappe <79846863+ilappe@users.noreply.github.com>
Co-authored-by: ilappe <ilappe@paloaltonetworks.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Community Support Level Indicates that the contribution is for Community supported pack Contribution Thank you! Contributions are always welcome! External PR pending-contributor The PR is pending the response of its creator post-demo ready-for-instance-test In contribution PRs, this label will cause a trigger of a build with a modified pack from the PR.
Projects
None yet
5 participants