Skip to content

Commit

Permalink
The match option is required for the email output plugin
Browse files Browse the repository at this point in the history
Whilst modifying the plugin to support sendmail delivery
the requirement for the match option was removed.

The change was made in this commit:

ef28018

This causes the email plugin to fail with the following error:

undefined method `each' for nil:NilClass

Tracing back to:

/opt/logstash/logstash.jar!/logstash/outputs/email.rb:173:in `receive'"

This commit reverts that change and marks the match option as
required.
  • Loading branch information
jamtur01 committed Dec 28, 2012
1 parent 3132c2a commit 1e02e98
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/logstash/outputs/email.rb
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ class LogStash::Outputs::Email < LogStash::Outputs::Base

# the registered fields that we want to monitor
# A hash of matches of field => value
config :match, :validate => :hash
config :match, :validate => :hash, :required => true

# the To address setting - fully qualified email address to send to
config :to, :validate => :string, :required => true
Expand Down

0 comments on commit 1e02e98

Please sign in to comment.