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

logstash-forwarder crashes with a divede-by-zero panic if the servers section is empty #455

Open
igalic opened this issue May 6, 2015 · 1 comment
Open

logstash-forwarder crashes with a divede-by-zero panic if the servers section is empty #455

igalic opened this issue May 6, 2015 · 1 comment
Labels
bug filebeat

Comments

@igalic
Copy link
Contributor

igalic commented May 6, 2015

this should be caught, as it's not really helpful.
especially given that we already get a nice error message when the files section is empty.
@igalic
Copy link
Contributor Author

igalic commented May 6, 2015

/cc @antaflos for gdb traces :)

magnusbaeck added a commit to magnusbaeck/logstash-forwarder that referenced this issue Jul 25, 2015
@magnusbaeck
Detect empty server list in config to avoid division by zero.
ae1a422 
Check that the merged configuration files define at least one
destination server so that we don't panic with a division by
zero error later on when trying to use the empty list of servers.
This fixes issue elastic#455.
magnusbaeck added a commit to magnusbaeck/logstash-forwarder that referenced this issue Jul 25, 2015
@magnusbaeck
Detect empty server list in config to avoid division by zero.
bb58202 
Check that the merged configuration files define at least one
destination server so that we don't panic with a division by
zero error later on when trying to use the empty list of servers.
This fixes issue elastic#455.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug filebeat
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jordansissel @ruflin @igalic