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] Akamai WAF - Content Pack Update #25457

Conversation

xsoar-bot
Copy link
Contributor

Status

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

Contributor

@davistonehub

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 the Contribution Thank you! Contributions are always welcome! label Mar 22, 2023
@content-bot content-bot changed the base branch from master to contrib/xsoar-contrib_davistonehub-contrib-Akamai_WAF March 22, 2023 14:27
@content-bot
Copy link
Collaborator

Thank you for your contribution. Your generosity and caring are unrivaled! Rest assured - our content wizard @daryakoval will very shortly look over your proposed changes.

@daryakoval daryakoval requested review from yucohen and removed request for daryakoval March 22, 2023 15:17
@daryakoval daryakoval assigned yucohen and unassigned daryakoval Mar 22, 2023
@gal-forer gal-forer requested review from gal-forer and removed request for yucohen March 22, 2023 18:43
@gal-forer gal-forer assigned gal-forer and unassigned yucohen Mar 22, 2023
Packs/Akamai_WAF/Integrations/Akamai_WAF/Akamai_WAF.py Outdated Show resolved Hide resolved
Packs/Akamai_WAF/Integrations/Akamai_WAF/Akamai_WAF.py Outdated Show resolved Hide resolved
Packs/Akamai_WAF/Integrations/Akamai_WAF/Akamai_WAF.py Outdated Show resolved Hide resolved
Packs/Akamai_WAF/Integrations/Akamai_WAF/Akamai_WAF.py Outdated Show resolved Hide resolved
Packs/Akamai_WAF/ReleaseNotes/2_1_0.md Outdated Show resolved Hide resolved
@gal-forer gal-forer added the pending-contributor The PR is pending the response of its creator label Mar 25, 2023
@gal-forer
Copy link
Contributor

@davistonehub Hi, thank you for the contribution, I would love for us to have a demo for the new commands, please let me know when you're available ? You can reach me at gforer@paloaltonetworks.com and I'm also on DFIR

@gal-forer
Copy link
Contributor

@davistonehub And also because this is an XSOAR supported integration you'll have to add unit tests to the new commands

name: change_path
required: true
description: acknowledge pre verification warning
name: akamai-acknowledge-pre-verification-warning
dockerimage: demisto/akamai:1.0.0.48112
- arguments:
- description: Unique identifier for the contract
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
- description: Unique identifier for the contract
- description: Unique identifier for the contract.

required: true
- description: name of the papi property
name: property_name
- description: Unique identifier for the group
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
- description: Unique identifier for the group
- description: Unique identifier for the group.

Packs/Akamai_WAF/Integrations/Akamai_WAF/Akamai_WAF.yml Outdated Show resolved Hide resolved
Packs/Akamai_WAF/Integrations/Akamai_WAF/Akamai_WAF.yml Outdated Show resolved Hide resolved
Packs/Akamai_WAF/Integrations/Akamai_WAF/Akamai_WAF.yml Outdated Show resolved Hide resolved
Packs/Akamai_WAF/Integrations/Akamai_WAF/Akamai_WAF.yml Outdated Show resolved Hide resolved
Packs/Akamai_WAF/Integrations/Akamai_WAF/Akamai_WAF.yml Outdated Show resolved Hide resolved
Packs/Akamai_WAF/Integrations/Akamai_WAF/Akamai_WAF.yml Outdated Show resolved Hide resolved
Packs/Akamai_WAF/Integrations/Akamai_WAF/Akamai_WAF.yml Outdated Show resolved Hide resolved
- Updated the function "list_papi_property_bygroup_ec" to return more context values.
- Updated the command "get_appsec_config_latest_version_command" to return an error if the security configuration with the name provided is not found.
- Added a new command "list_papi_property_bygroup_command": Lists properties available for the current contract and group.
- Added a new command "get_papi_property_byname_command": Get papi property info without default rule. To get default rule, please use "get-papi-property-rule" command.
Copy link
Contributor

Choose a reason for hiding this comment

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

Delete the lines below since I added them as second level bullets in the previous comment. It's just easier to read that way.

Copy link
Contributor

Choose a reason for hiding this comment

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

Removed as per suggestion.

@ShirleyDenkberg
Copy link
Contributor

@gal-forer Doc review completed.

davistonehub and others added 5 commits March 28, 2023 13:21
Co-authored-by: gal-forer <forer.gal@gmail.com>
Co-authored-by: gal-forer <forer.gal@gmail.com>
Co-authored-by: gal-forer <forer.gal@gmail.com>
Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>
Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>
davistonehub and others added 13 commits March 28, 2023 13:23
Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>
Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>
Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>
Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>
Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>
Removed two comment lines 5040/5041 as per suggestion.
Remove line 6,9,10,11,12 as per suggestion
Change command name from "akami-list-papi-property-bygroup" to "akami-list-papi-property-by-group".
Change command name from "akami-get-papi-property-byname" to "akami-get-papi-property-by-name".
Change command name from "akami-get-papi-property-byid" to "akami-get-papi-property-by-id".
Changed the name of the command from "list_papi_property_bygroup_command" to "list_papi_property_by_group_command".
Changed the name of the command from "get_papi_property_byname_command" to "get_papi_property_by_name_command".
Changed the name of the command from "get_papi_property_byid_command" to " get_papi_property_by_id_command".
Corrected the variable "entry_context" assignment in command "list_papi_property_by_group_command".
Allowed the non-case-sensitive comparison in command "clone_security_policy_command" by adding ".lower()" function.
replace "proerty" with "property"
Fixed indents.
Changed line "list papi property bygroup command" to "list papi property by group command".
Changed line "get papi property byname command" to "get papi property by name command".
Changed line "get papi property byid command" to "get papi property by id command".
@gal-forer gal-forer added ready-for-instance-test In contribution PRs, this label will cause a trigger of a build with a modified pack from the PR. post-demo labels Apr 1, 2023
@content-bot
Copy link
Collaborator

content-bot commented Apr 1, 2023

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

@github-advanced-security
Copy link

You have successfully added a new CodeQL configuration /language:javascript. As part of the setup process, we have scanned this repository and found 10 existing alerts. Please check the repository Security tab to see all alerts.

@github-advanced-security
Copy link

You have successfully added a new CodeQL configuration /language:python. As part of the setup process, we have scanned this repository and found 78 existing alerts. Please check the repository Security tab to see all alerts.

@gal-forer gal-forer merged commit c743bcc into demisto:contrib/xsoar-contrib_davistonehub-contrib-Akamai_WAF Apr 4, 2023
18 of 21 checks passed
DeanArbel pushed a commit that referenced this pull request Apr 4, 2023
* [Marketplace Contribution] Akamai WAF - Content Pack Update (#25457)

* "contribution update to pack "Akamai WAF""

* Update 2_1_0.md

* Update Packs/Akamai_WAF/Integrations/Akamai_WAF/Akamai_WAF.py

Co-authored-by: gal-forer <forer.gal@gmail.com>

* Update Packs/Akamai_WAF/Integrations/Akamai_WAF/Akamai_WAF.py

Co-authored-by: gal-forer <forer.gal@gmail.com>

* Update Packs/Akamai_WAF/Integrations/Akamai_WAF/Akamai_WAF.py

Co-authored-by: gal-forer <forer.gal@gmail.com>

* Update Packs/Akamai_WAF/Integrations/Akamai_WAF/Akamai_WAF.yml

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>

* Update Packs/Akamai_WAF/Integrations/Akamai_WAF/Akamai_WAF.yml

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>

* Update Packs/Akamai_WAF/Integrations/Akamai_WAF/Akamai_WAF.yml

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>

* Update Packs/Akamai_WAF/Integrations/Akamai_WAF/Akamai_WAF.yml

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>

* Update Packs/Akamai_WAF/Integrations/Akamai_WAF/Akamai_WAF.yml

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>

* Update Packs/Akamai_WAF/Integrations/Akamai_WAF/Akamai_WAF.yml

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>

* Update Packs/Akamai_WAF/Integrations/Akamai_WAF/Akamai_WAF.yml

Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>

* Update Akamai_WAF.py

Removed two comment lines 5040/5041 as per suggestion.

* Update 2_1_0.md

Remove line 6,9,10,11,12 as per suggestion

* Update Akamai_WAF.yml

Change command name from "akami-list-papi-property-bygroup" to "akami-list-papi-property-by-group".
Change command name from "akami-get-papi-property-byname" to "akami-get-papi-property-by-name".
Change command name from "akami-get-papi-property-byid" to "akami-get-papi-property-by-id".

* Update Akamai_WAF.py

Changed the name of the command from "list_papi_property_bygroup_command" to "list_papi_property_by_group_command".
Changed the name of the command from "get_papi_property_byname_command" to "get_papi_property_by_name_command".
Changed the name of the command from "get_papi_property_byid_command" to " get_papi_property_by_id_command".
Corrected the variable "entry_context" assignment in command "list_papi_property_by_group_command".
Allowed the non-case-sensitive comparison in command "clone_security_policy_command" by adding ".lower()" function.

* Update Akamai_WAF.py

replace "proerty" with "property"

* Update Akamai_WAF.py

Fixed indents.

* Update Akamai_WAF.py

Changed line "list papi property bygroup command" to "list papi property by group command".
Changed line "get papi property byname command" to "get papi property by name command".
Changed line "get papi property byid command" to "get papi property by id command".

---------

Co-authored-by: davistonehub <111578758+davistonehub@users.noreply.github.com>
Co-authored-by: gal-forer <forer.gal@gmail.com>
Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>

* Regenerated readme

* Regenerated readme

---------

Co-authored-by: xsoar-bot <67315154+xsoar-bot@users.noreply.github.com>
Co-authored-by: davistonehub <111578758+davistonehub@users.noreply.github.com>
Co-authored-by: gal-forer <forer.gal@gmail.com>
Co-authored-by: ShirleyDenkberg <62508050+ShirleyDenkberg@users.noreply.github.com>
Co-authored-by: gal-forer <gforer@paloaltonetworks.com>
@xsoar-bot xsoar-bot deleted the davistonehub-contrib-Akamai_WAF branch November 23, 2023 16:13
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! docs-approved 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
7 participants