Skip to content

CRTX-261255 - fix FortiGate os_family iOS misclassification - #44966

Merged
eepstain merged 5 commits into
masterfrom
CRTX-261255-fix-fortigate-os-family-ios
Jul 9, 2026
Merged

CRTX-261255 - fix FortiGate os_family iOS misclassification#44966
eepstain merged 5 commits into
masterfrom
CRTX-261255-fix-fortigate-os-family-ios

Conversation

@eepstain

@eepstain eepstain commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Related Issues

https://jira-dc.paloaltonetworks.com/browse/CRTX-261255

Description

Fixed an issue where FortiGate events were incorrectly assigned the iOS OS family. The xdm.source.host.os_family and xdm.target.host.os_family mappings used contains "ios", which matched the substring ios inside FortiOS (the FortiGate operating system). As a result, stitched story (DML) events from FortiGate were labeled with os_family = IOS. The mapping now normalizes the value with lowercase() and compares it exactly (lowercase(src_os) = "ios" / lowercase(dst_os) = "ios"), so genuine iOS logs still resolve correctly while fortios no longer collides.

@eepstain eepstain added docs-approved 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 Jul 6, 2026
@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

@eepstain
eepstain force-pushed the CRTX-261255-fix-fortigate-os-family-ios branch from 55cab75 to 4a9a7f7 Compare July 6, 2026 12:06
@eepstain
eepstain requested a review from sdaniel6 July 6, 2026 12:12
@content-bot

This comment has been minimized.

@eepstain eepstain added research-mapping-approve This changes is belongs to research mapping. se-approved Pack available in SE labels Jul 7, 2026
@content-bot

This comment has been minimized.

@eepstain
eepstain force-pushed the CRTX-261255-fix-fortigate-os-family-ios branch from 5365d26 to 4a9a7f7 Compare July 7, 2026 13:59
@eepstain
eepstain force-pushed the CRTX-261255-fix-fortigate-os-family-ios branch from 4a9a7f7 to 59da3e8 Compare July 7, 2026 14:39
@content-bot

This comment has been minimized.

@content-bot

This comment has been minimized.

@content-bot

This comment has been minimized.

@content-bot

Copy link
Copy Markdown
Contributor

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? ✅

@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: 4c505d2a01621626

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

⚠️ AI-generated triage. Validate before acting.

@eepstain
eepstain merged commit ff59c80 into master Jul 9, 2026
25 checks passed
@eepstain
eepstain deleted the CRTX-261255-fix-fortigate-os-family-ios branch July 9, 2026 11:52
anyrun-integrations pushed a commit to anyrun-integrations/cortex_xsoar that referenced this pull request Aug 27, 2026
…44966)

* CRTX-261255 - fix FortiGate os_family iOS misclassification

* Trigger GitHub pipeline (user-created PR)

* Trigger GitHub pipeline (user-created PR)

* Trigger GitHub pipeline (user-created PR)

---------

Co-authored-by: CI Bot <ci@demisto.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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 research-mapping-approve This changes is belongs to research mapping. se-approved Pack available in SE skip-ai-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants