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.

Sign up for GitHub

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

Jump to bottom

Add max_line_size option #29

Closed
andrewvc opened this issue Jun 24, 2016 · 0 comments · Fixed by #43 · May be fixed by #30
Closed

Add max_line_size option #29

andrewvc opened this issue Jun 24, 2016 · 0 comments · Fixed by #43 · May be fixed by #30
Assignees
@andsel
Labels
enhancement P2

Comments

@andrewvc
Copy link
Contributor

It is easy to OOM logstash with lines that are too long. This can happen if the user inadvertently sends a very large file to logstash without the right delimiter. We should set a default of perhaps 20MB for a max document line size and raise a specific error if that happens.
andrewvc added a commit to andrewvc/logstash-codec-json_lines that referenced this issue Aug 5, 2016
@andrewvc
Add decode_size_limit_bytes option.
f5e1872 
Resolves logstash-plugins#29
@andrewvc andrewvc mentioned this issue Aug 5, 2016
Open
andsel pushed a commit to andsel/logstash-codec-json_lines that referenced this issue Aug 27, 2024
@andrewvc @andsel
Add decode_size_limit_bytes option.
d0ed42d 
Resolves logstash-plugins#29
@andsel andsel linked a pull request Aug 28, 2024 that will close this issue
Size limit bytes #43
Merged
4 tasks
@andsel andsel mentioned this issue Aug 28, 2024
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@andsel andsel

Labels
enhancement P2
Projects
None yet
Milestone
No milestone
2 participants
@andrewvc @andsel