elastic / elastic/logstash

LogStash::Event should not allow nil keys!

Open
#4,105 20 comments 0 reactions 0 assignees View on GitHub
bug crashes design
Dominant language
Java
Stars
14.9k
Forks
3.5k
Avg merge
19h 14m
Merged PRs (30d)
63

Description

Currently, it is possible to create an event with nil keys using the following code `LogStash::Event.new(nil => 123)`. This should fail! It really screws with things like the Elasticsearch output.

The accessor `[]` does not have this error, but should raise a better error message (it just happens to fail by trying to invoke `nil.starts_with?`.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.