Skip to content

zscaler_zia: fix reversed source.bytes/destination.bytes in firewall - #19463

Merged
efd6 merged 1 commit into
elastic:mainfrom
efd6:s7266-zscaler_zia
Jun 10, 2026
Merged

zscaler_zia: fix reversed source.bytes/destination.bytes in firewall#19463
efd6 merged 1 commit into
elastic:mainfrom
efd6:s7266-zscaler_zia

Conversation

@efd6

@efd6 efd6 commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Proposed commit message

zscaler_zia: fix reversed source.bytes/destination.bytes in firewall

The firewall ingest pipeline mapped Zscaler's inbytes field to
source.bytes and outbytes to destination.bytes. Zscaler defines
inbytes as server-to-client and outbytes as client-to-server, so
with the integration's existing IP mapping (csip/client = source),
the byte fields were reversed.

Swap the ECS targets so outbytes maps to source.bytes and inbytes
maps to destination.bytes. This is a breaking change: dashboards,
alerts, or detection rules referencing source.bytes or
destination.bytes from firewall logs will see swapped values.

ref: https://help.zscaler.com/zia/nss-feed-output-format-firewall-logs

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots

@efd6 efd6 self-assigned this Jun 9, 2026
@efd6 efd6 added Integration:zscaler_zia Zscaler Internet Access bugfix Pull request that fixes a bug issue Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] labels Jun 9, 2026
@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

✅ Elastic Docs Style Checker (Vale)

No issues found on modified lines!


The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale.

@efd6
efd6 force-pushed the s7266-zscaler_zia branch 2 times, most recently from 3a2fdf9 to af422c0 Compare June 9, 2026 21:29
@efd6
efd6 marked this pull request as ready for review June 9, 2026 22:16
@efd6
efd6 requested review from a team as code owners June 9, 2026 22:16
@infra-vault-gh-plugin-prod

Copy link
Copy Markdown

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

The firewall ingest pipeline mapped Zscaler's inbytes field to
source.bytes and outbytes to destination.bytes. Zscaler defines
inbytes as server-to-client and outbytes as client-to-server, and
Zscaler's own field name prefixes encode the intended ECS mapping:
cs (client-source), cd (client-destination), ss (server-source),
sd (server-destination). The integration already follows this for
IP fields (csip -> source.ip), but the byte fields were reversed.

Swap the ECS targets so outbytes maps to source.bytes and inbytes
maps to destination.bytes, consistent with the IP mapping and with
Zscaler's naming convention. This is a breaking change: dashboards,
alerts, or detection rules referencing source.bytes or
destination.bytes from firewall logs will see swapped values.

ref: https://help.zscaler.com/zia/nss-feed-output-format-firewall-logs
@efd6
efd6 force-pushed the s7266-zscaler_zia branch from af422c0 to a8d103e Compare June 10, 2026 03:45
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

✅ All changelog entries have the correct PR link.

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @efd6

@navnit-elastic navnit-elastic left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@efd6
efd6 merged commit be92ae3 into elastic:main Jun 10, 2026
9 checks passed
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package zscaler_zia - 4.0.0 containing this change is available at https://epr.elastic.co/package/zscaler_zia/4.0.0/

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

Labels

bugfix Pull request that fixes a bug issue Integration:zscaler_zia Zscaler Internet Access Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

3 participants