Skip to content

Update ecslog to fix an infinite loop#15142

Merged
ph merged 1 commit intoelastic:fleetfrom
ph:agent/update-ecslog
Dec 17, 2019
Merged

Update ecslog to fix an infinite loop#15142
ph merged 1 commit intoelastic:fleetfrom
ph:agent/update-ecslog

Conversation

@ph
Copy link
Copy Markdown
Contributor

@ph ph commented Dec 17, 2019

This PR add the fix from urso/ecslog#17
Which fix the infinite loop when using parsing flags, the bug was making
the library consume all the available memory until the OS killed the
process.

This PR add the fix from urso/ecslog#17
Which fix the infinite loop when using parsing flags, the bug was making
the library consume all the available memory until the OS killed the
process.
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ingest (Project:fleet)

@ph ph requested a review from michalpristas December 17, 2019 13:24
Copy link
Copy Markdown
Contributor

@michalpristas michalpristas left a comment

Choose a reason for hiding this comment

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

LGTM

@ph
Copy link
Copy Markdown
Contributor Author

ph commented Dec 17, 2019

going to merge it, we target a branch and the failures should not impact the code.

@ph ph merged commit 38591b9 into elastic:fleet Dec 17, 2019
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
This PR add the fix from urso/ecslog#17
Which fix the infinite loop when using parsing flags, the bug was making
the library consume all the available memory until the OS killed the
process.
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.

3 participants