Skip to content

Packs modifications persistence ya #41966

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Packs modifications persistence ya #41966

wants to merge 4 commits into from
+30 −9

Conversation

@yasta5
Copy link
Contributor

@yasta5 yasta5 commented Nov 19, 2025 •
edited
Loading

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)

Description

Modified 3 packs to align with persistence:

  • SymantecBlueCoatProxySG - Replace modification of system field.
  • CloudflareWAF - Added fields to schema file.
  • Prisma Cloud Compute - Modified parsing_rule to be enable time field processing.

@yasta5 yasta5 requested a review from OBavly November 19, 2025 11:14
@yasta5 yasta5 added bypass.url Whether to create build bucket, add this label for marketplace.bootstrap.bypass.url ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines labels Nov 19, 2025
@content-bot
Copy link
Collaborator

content-bot commented Nov 19, 2025

Validate summary
The following errors were thrown as a part of this pr: .
If the AG100 validation in the pre-commit GitHub Action fails, the pull request cannot be force-merged.

Verdict: PR can be force merged from validate perspective? ✅

@talihaff
Copy link
Contributor

talihaff commented Nov 19, 2025

@OBavly docs approved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@OBavly OBavly Awaiting requested review from OBavly

At least 1 approving review is required to merge this pull request.

Assignees

@talihaff talihaff

Labels

bypass.url Whether to create build bucket, add this label for marketplace.bootstrap.bypass.url docs-approved ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@yasta5 @content-bot @talihaff @richardbluestone