Skip to content

fix [panw]: Increase allowed characters in hostname and username parsing - #9479

Merged
mjwolf merged 3 commits into
elastic:mainfrom
mjwolf:panw-hostname
Apr 2, 2024
Merged

fix [panw]: Increase allowed characters in hostname and username parsing#9479
mjwolf merged 3 commits into
elastic:mainfrom
mjwolf:panw-hostname

Conversation

@mjwolf

@mjwolf mjwolf commented Apr 1, 2024

Copy link
Copy Markdown
Contributor

Proposed commit message

For some users of panw, datastream parsing has been failing as they use characters in hostname and username fields that weren't being parsed.

This adds support for additional patterns in these fields:

  • Hostnames with '_' characters
  • Usernames with '#' characters

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.

Author's Checklist

How to test this PR locally

Related issues

Screenshots

…astream

For some users of panw, datastream parsing has been failing as they use
characters in hostname and username fields that weren't being parsed.

This adds support for additional patterns in these fields:
* Hostnames with '_' characters
* Usernames with '#' characters
@mjwolf mjwolf added the bugfix Pull request that fixes a bug issue label Apr 1, 2024
@mjwolf
mjwolf requested a review from a team as a code owner April 1, 2024 19:11
@mjwolf

mjwolf commented Apr 1, 2024

Copy link
Copy Markdown
Contributor Author

/test

@mjwolf
mjwolf enabled auto-merge (squash) April 1, 2024 19:36
Comment thread packages/panw/data_stream/panos/elasticsearch/ingest_pipeline/default.yml Outdated
Comment thread packages/panw/data_stream/panos/elasticsearch/ingest_pipeline/default.yml Outdated
@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

@mjwolf
mjwolf merged commit 51a6c21 into elastic:main Apr 2, 2024
@elastic-sonarqube

Copy link
Copy Markdown
@elasticmachine

Copy link
Copy Markdown

Package panw - 3.24.1 containing this change is available at https://epr.elastic.co/search?package=panw

@mjwolf
mjwolf deleted the panw-hostname branch April 2, 2024 16:57
@andrewkroh andrewkroh added the Integration:panw Palo Alto Next-Gen Firewall label Jul 22, 2024
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:panw Palo Alto Next-Gen Firewall

4 participants