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

Cherry-pick #9869 to 6.6: Handle IPv6 zone id in IIS filebeat ingest pipeline #9933

Merged
merged 2 commits into from
Jan 9, 2019

Conversation

jsoriano
Copy link
Member

@jsoriano jsoriano commented Jan 7, 2019

Cherry-pick of PR #9869 to 6.6 branch. Original message:

Possible workaround for #9836, this may be needed in other modules if other services also log IPv6 addresses with zone id.

Wait for elastic/elasticsearch#37107 before deciding what to do about this.

IIS logs can include zone ids when using IPv6, this is correctly parsed
but geoip processor doesn't accept these addresses. Create a temporary
field without the zone id to be used by geoip processor.

(cherry picked from commit d59ae8c)
@ruflin
Copy link
Member

ruflin commented Jan 8, 2019

You need to regenerate the iis files as they look a bit different in 6.6.

@jsoriano
Copy link
Member Author

jsoriano commented Jan 8, 2019

Expected JSON regenerated

@jsoriano
Copy link
Member Author

jsoriano commented Jan 8, 2019

Failing test is #9215

@jsoriano jsoriano merged commit 5f1f6ca into elastic:6.6 Jan 9, 2019
@jsoriano jsoriano deleted the backport_9869_6.6 branch January 9, 2019 12:46
jsoriano added a commit to jsoriano/beats that referenced this pull request Jan 14, 2019
…ingest pipeline (elastic#9933)

IIS logs can include zone ids when using IPv6, this is correctly parsed
but geoip processor doesn't accept these addresses. Create a temporary
field without the zone id to be used by geoip processor.

(cherry picked from commit d59ae8c)
(cherry picked from commit 5f1f6ca)
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
…ingest pipeline (elastic#9933)

IIS logs can include zone ids when using IPv6, this is correctly parsed
but geoip processor doesn't accept these addresses. Create a temporary
field without the zone id to be used by geoip processor.

(cherry picked from commit dad84e9)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants