Skip to content

Remove accidental overwrite of user_agent.device.name by user_agent.device#10472

Merged
webmat merged 4 commits intoelastic:masterfrom
webmat:ecs-fix-ua-device-name
Feb 1, 2019
Merged

Remove accidental overwrite of user_agent.device.name by user_agent.device#10472
webmat merged 4 commits intoelastic:masterfrom
webmat:ecs-fix-ua-device-name

Conversation

@webmat
Copy link
Copy Markdown
Contributor

@webmat webmat commented Jan 31, 2019

This is a cleanup that can and should go in before #10441.

@webmat webmat requested review from a team as code owners January 31, 2019 19:52
@webmat webmat self-assigned this Jan 31, 2019
Copy link
Copy Markdown
Contributor Author

@webmat webmat Jan 31, 2019

Choose a reason for hiding this comment

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

This field def overrides the real ECS field, defined on line 2180

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

:-(

@webmat webmat requested a review from ruflin January 31, 2019 19:55
@webmat webmat force-pushed the ecs-fix-ua-device-name branch from 801c01e to c5f03c5 Compare January 31, 2019 19:59
@webmat webmat requested a review from a team as a code owner January 31, 2019 20:16
Copy link
Copy Markdown
Contributor

@ruflin ruflin left a comment

Choose a reason for hiding this comment

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

Good to be merged, CI failure is not related.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

:-(

@webmat webmat force-pushed the ecs-fix-ua-device-name branch from 48e4ed4 to f7b6f07 Compare February 1, 2019 13:23
@webmat
Copy link
Copy Markdown
Contributor Author

webmat commented Feb 1, 2019

jenkins, test this

@webmat
Copy link
Copy Markdown
Contributor Author

webmat commented Feb 1, 2019

Yep, test failures still unrelated after rebase. Merging

@webmat webmat merged commit c57c9ee into elastic:master Feb 1, 2019
@webmat webmat deleted the ecs-fix-ua-device-name branch February 1, 2019 21:49
DStape pushed a commit to DStape/beats that referenced this pull request Aug 20, 2019
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