Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Apache.access] Changed source and destination fields in a backward-compatible manner (ECS) #10416

Closed

Conversation

Alphayeeeet
Copy link
Contributor

@Alphayeeeet Alphayeeeet commented Jul 8, 2024

Please label as enhancement!

Proposed commit message

Changed source and destination fields in a backward compatible manner

Changed destination.domain to .address and added processor to split into .domain and .ip. Also added source/destination.port fields. This now allows support for parsing server-host and port as well as client-host and port from the log.

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.

Sorry, something went wrong.

…nto .domain and .ip. Also added source/destination.port fields.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@Alphayeeeet Alphayeeeet requested a review from a team as a code owner July 8, 2024 19:16
@Alphayeeeet Alphayeeeet changed the title Changed source and destination fields in a backward-compatible manner (ECS) [Apache.access] Changed source and destination fields in a backward-compatible manner (ECS) Jul 8, 2024
@Alphayeeeet Alphayeeeet marked this pull request as draft July 9, 2024 15:43
@Alphayeeeet Alphayeeeet marked this pull request as ready for review July 16, 2024 09:07
@andrewkroh andrewkroh added Integration:apache Apache HTTP Server Team:Obs-InfraObs Label for the Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations] labels Jul 19, 2024
@Alphayeeeet
Copy link
Contributor Author

This PR has already been implemented so far by #10228

@Alphayeeeet Alphayeeeet deleted the apache-access-add-host-fields branch July 22, 2024 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Integration:apache Apache HTTP Server Team:Obs-InfraObs Label for the Observability Infrastructure Monitoring team [elastic/obs-infraobs-integrations]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants