Skip to content

Conversation

mjwolf
Copy link
Contributor

@mjwolf mjwolf commented Oct 8, 2025

Proposed commit message

In 302xxx messages, add properly parse IPv6 addresses. Previously, the address could be improperly parsed into interface, as both use colon seperators. This reorders the parsing to do IPv6 first, so that these addresses will be correctly parsed, while still parsing the interface if the field isn't IPv6.

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

In 302xxx messages, add properly parse IPv6 addresses. Previously, the
address could be improperly parsed into interface, as both use colon
seperators. This reorders the parsing to do IPv6 first, so that these
addresses will be correctly parsed, while still parsing the interface if
the field isn't IPv6.
@mjwolf mjwolf requested a review from a team as a code owner October 8, 2025 19:22
@mjwolf mjwolf added Integration:cisco_asa Cisco ASA bugfix Pull request that fixes a bug issue Team:Integration-Experience Security Integrations Integration Experience [elastic/integration-experience] labels Oct 8, 2025
@elasticmachine
Copy link

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

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

elastic-vault-github-plugin-prod bot commented Oct 8, 2025

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@mjwolf mjwolf enabled auto-merge (squash) October 12, 2025 19:01
@elasticmachine
Copy link

💚 Build Succeeded

History

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:cisco_asa Cisco ASA Team:Integration-Experience Security Integrations Integration Experience [elastic/integration-experience]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants