Skip to content

Sk csp control optional param - #45613

Draft
shmuel44 wants to merge 4 commits into
masterfrom
sk_csp_control_optional_param
Draft

Sk csp control optional param#45613
shmuel44 wants to merge 4 commits into
masterfrom
sk_csp_control_optional_param

Conversation

@shmuel44

Copy link
Copy Markdown
Contributor

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

Description

Must have

  • Tests
  • Documentation

Control case for CIAC-17274 AI-adjudication testing. The new parameter has a default and does not change behavior, so every existing call site remains valid. Expected verdict is safe. Disposable test branch, MUST NOT be merged.
@content-bot

content-bot commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

CommonServerPython dependency impact

Impacted content items were found and need manual verification.

  • Changed symbols: 1
  • Public entrypoints: 1
  • Affected files: 6
  • Affected packs: 5

PaloAltoNetworks_IoT3rdParty

  • Packs/PaloAltoNetworks_IoT3rdParty/Integrations/PaloAltoNetworksIoT3rdParty/PaloAltoNetworksIoT3rdParty.py - matched: is_mac_address
  • Packs/PaloAltoNetworks_IoT3rdParty/Scripts/SendAllPANWIoTDevicesToCiscoISE/SendAllPANWIoTDevicesToCiscoISE.py - matched: is_mac_address

FortiGate

  • Packs/FortiGate/Integrations/FortiGate/FortiGate.py - matched: is_mac_address

GoogleChronicleBackstory

  • Packs/GoogleChronicleBackstory/Scripts/ListDeviceEventsScript/ListDeviceEventsScript.py - matched: is_mac_address

Lansweeper

  • Packs/Lansweeper/Integrations/Lansweeper/Lansweeper.py - matched: is_mac_address

cisco-ise

  • Packs/cisco-ise/Integrations/cisco-ise/cisco-ise.py - matched: is_mac_address

Open the HTML impact checklist

To accept this change without verifying each item, apply one of these PR labels: I'm okay with it, not-relevant-for-csp-validation.

Analyzed commit 36e93d21 · generated 2026-08-23T20:34:19Z · CI job

@content-bot

Copy link
Copy Markdown
Contributor

🤖 AI-Powered Code Review Available

You can leverage AI-powered code review to assist with this PR!

Available Commands:

  • @marketplace-ai-reviewer start review - Initiate a full AI code review
  • @marketplace-ai-reviewer re-review - Incremental review for new commits

@github-actions

Copy link
Copy Markdown

⚠️ CommonServerPython change detected

This PR modifies one of the runtime-injected helper scripts (CommonServerPython.py / CommonServerPowerShell.ps1 / CommonServer.js).
A change here can impact every integration and script in the repository, so the standard PR pipeline is not enough.

Required before merge:

  1. Trigger the content nightly pipeline against this branch.
  2. Make sure the nightly run is green (all jobs passed).
  3. Add the nightly-run-passed label to this PR so this check turns green.

If you believe nightly is not required for this change, please justify it in a PR comment and add the label to unblock the check.

@github-actions

github-actions Bot commented Aug 23, 2026

Copy link
Copy Markdown

Coverage

Tests Skipped Failures Errors Time
7287 269 💤 0 ❌ 0 🔥 5m 41s ⏱️

@content-bot

This comment has been minimized.

@content-bot

content-bot commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

CommonServerPython dependency impact

The AI adjudication stage found no breaking change.

Adjudicated: 0 breaking, 0 warning, 6 safe.

Impacted content items were found and need manual verification.

  • Changed symbols: 1
  • Public entrypoints: 1
  • Affected files: 6
  • Affected packs: 5

PaloAltoNetworks_IoT3rdParty

  • Packs/PaloAltoNetworks_IoT3rdParty/Integrations/PaloAltoNetworksIoT3rdParty/PaloAltoNetworksIoT3rdParty.py - matched: is_mac_address
  • Packs/PaloAltoNetworks_IoT3rdParty/Scripts/SendAllPANWIoTDevicesToCiscoISE/SendAllPANWIoTDevicesToCiscoISE.py - matched: is_mac_address

FortiGate

  • Packs/FortiGate/Integrations/FortiGate/FortiGate.py - matched: is_mac_address

GoogleChronicleBackstory

  • Packs/GoogleChronicleBackstory/Scripts/ListDeviceEventsScript/ListDeviceEventsScript.py - matched: is_mac_address

Lansweeper

  • Packs/Lansweeper/Integrations/Lansweeper/Lansweeper.py - matched: is_mac_address

cisco-ise

  • Packs/cisco-ise/Integrations/cisco-ise/cisco-ise.py - matched: is_mac_address

Open the HTML impact checklist

To accept this change without verifying each item, apply one of these PR labels: I'm okay with it, not-relevant-for-csp-validation.

Analyzed commit 36e93d21 · generated 2026-08-23T20:34:19Z · CI job

@content-bot

This comment has been minimized.

@content-bot

Copy link
Copy Markdown
Contributor

⚠️ The PR is missing the ready-for-pipeline-running label. Please add the label when the PR is ready in order to proceed.

@content-bot

Copy link
Copy Markdown
Contributor

Validate summary
The following errors were thrown as a part of this pr: RN106, PA114.
The following errors cannot be ignored: RN106, PA114.
The following errors don't run as part of the nightly flow and therefore can be force merged: RN106, PA114.
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? ✅

@content-bot

Copy link
Copy Markdown
Contributor

🔍 AI Triage Report Available

An automated triage report has been generated for this pipeline.

Status: failed
Report ID: 6b879bb02e687e98

📋 Triage Report
💡 Resolutions are available in the full report.

⚠️ AI-generated triage. Validate before acting.

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.

2 participants