Skip to content

[Citrix ADC] Add parsing for APPFW native messages with IP-based format#17367

Merged
haetamoudi merged 7 commits intoelastic:mainfrom
haetamoudi:6878-issue-parsing-netscaler-appfw-logs-as-either-cef-or-syslog-message
Feb 23, 2026
Merged

[Citrix ADC] Add parsing for APPFW native messages with IP-based format#17367
haetamoudi merged 7 commits intoelastic:mainfrom
haetamoudi:6878-issue-parsing-netscaler-appfw-logs-as-either-cef-or-syslog-message

Conversation

@haetamoudi
Copy link
Copy Markdown
Contributor

Fixes: https://github.com/elastic/sdh-beats/issues/6878

Proposed commit message

Add parsing for APPFW native messages with IP-based format

Fixes parsing issues with Citrix ADC Application Firewall (APPFW) logs from Netscaler ADC 14.1 in two scenarios:

  1. RFC5424 Native APPFW Messages
    Problem: APPFW_POLICY_HIT messages with IP-based format not fully parsed. Fields like source.ip, profile names, and URLs are unparsed in citrix_adc.log.message.
    Fix: Updated grok patterns in appfw_feature.yml to correctly extract missing fields from IP-based APPFW messages.

  2. CEF Messages in RFC5424 Syslog
    Problem: CEF messages wrapped in RFC5424 syslog headers routed to the CEF pipeline, leaving them unparsed in citrix.extended.message.
    Fix: Update default.yml to detect CEF content after native RFC5424 parsing and route it to the CEF pipeline. This adds support for CEF over syslog (not just file-based CEF).

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

Screenshots

@haetamoudi haetamoudi requested review from a team as code owners February 11, 2026 09:32
@haetamoudi haetamoudi added Integration:citrix_adc Citrix ADC bugfix Pull request that fixes a bug issue Team:Integration-Experience Security Integrations Integration Experience [elastic/integration-experience] labels Feb 11, 2026
@elasticmachine
Copy link
Copy Markdown

Pinging @elastic/integration-experience (Team:Integration-Experience)

@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

elastic-vault-github-plugin-prod Bot commented Feb 11, 2026

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

…mes, and URLs.

parse cef on top of native

update pr number

update values
@haetamoudi haetamoudi force-pushed the 6878-issue-parsing-netscaler-appfw-logs-as-either-cef-or-syslog-message branch from 9b59476 to 049894a Compare February 11, 2026 10:25
Copy link
Copy Markdown
Contributor

@jrmolin jrmolin left a comment

Choose a reason for hiding this comment

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

looks good to me

Copy link
Copy Markdown
Contributor

@ilyannn ilyannn left a comment

Choose a reason for hiding this comment

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

Looks good with minor questions

Comment thread packages/citrix_adc/changelog.yml
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 13, 2026

✅ Vale Linting Results

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.

@andrewkroh andrewkroh added the documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. label Feb 13, 2026
Comment thread packages/citrix_adc/changelog.yml Outdated
haetamoudi and others added 2 commits February 19, 2026 15:09
Co-authored-by: Taylor Swanson <90622908+taylor-swanson@users.noreply.github.com>
@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

History

@haetamoudi haetamoudi merged commit bcbff05 into elastic:main Feb 23, 2026
10 checks passed
haetamoudi added a commit to haetamoudi/integrations that referenced this pull request Feb 23, 2026
@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

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

navnit-elastic pushed a commit to navnit-elastic/integrations that referenced this pull request Mar 2, 2026
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 documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:citrix_adc Citrix ADC Team:Integration-Experience Security Integrations Integration Experience [elastic/integration-experience]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants