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

Backport #454 to 1.0: Format port numbers and numeric IDs as strings #467

Merged
merged 1 commit into from
May 24, 2019

Conversation

webmat
Copy link
Contributor

@webmat webmat commented May 23, 2019

Backport of PR #454 to 1.0 branch. Original message:

Changes the display format of things like port numbers and PIDs to string where appropriate. Changed fields are:

client.port
destination.port
event.severity
event.sequence (backport note: not in 1.0)
http.response.status_code
process.pid
process.ppid
process.pgid (backport note: not in 1.0)
process.thread.id
server.port
source.port
url.port

@webmat
Copy link
Contributor Author

webmat commented May 23, 2019

We'll need to merge this only after 1.0.1 has been released.

Related to this, 3 commits appear in this PR that are related to the pending 1.0.1 release. Only the latest commit is relevant in this backport PR.

Copy link
Member

@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.

LGTM, but needs rebase as soon as the 1.0.1 PR is merged.

Changes the display format of things like port numbers and PIDs to string where appropriate. Changed fields are:

client.port
destination.port
event.severity
event.sequence (cherry-pick note: not in 1.0)
http.response.status_code
process.pid
process.ppid
process.pgid (cherry-pick note: not in 1.0)
process.thread.id
server.port
source.port
url.port
@webmat webmat merged commit d72b609 into elastic:1.0 May 24, 2019
webmat pushed a commit to webmat/ecs that referenced this pull request May 24, 2019
…trings (elastic#467)

Backport of PR elastic#454 to 1.0 branch. Original message:

Changes the display format of things like port numbers and PIDs to string where appropriate. Changed fields are:

client.port
destination.port
event.severity
event.sequence (cherry-pick note: not in 1.0)
http.response.status_code
process.pid
process.ppid
process.pgid (cherry-pick note: not in 1.0)
process.thread.id
server.port
source.port
url.port
@webmat webmat added v1.0.1 and removed v1.0.2 labels May 24, 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