Allow level to be set for each event #1
Conversation
logstash-output-rollbar.gemspec
Outdated
There was a problem hiding this comment.
Seems like a breaking change; how about 0.2.0?
There was a problem hiding this comment.
Good point. Updated to 0.2.0 in 8bf3b32
|
Was just looking for exactly this behavior. Any reason not to also allow access tokens (and, for that matter, environment) to be set in this same way? I have a logstash configuration that takes in log files from a number of different services and I'd like to be able to dynamically pipe them to the appropriate Rollbar project. |
|
@goddardcm That's a great point. I really can't think of any reason not to just have everything work that way. In fact, I'm inclined to change "default_level" back to "level", consider that whole initial config to just be defaults, and let anything be overridden by a specific event. |
|
@kshep That seems reasonable to me. |
|
@goddardcm This should do it: a5507d7 |
|
Looks good to me! Thanks! |
Allow level to be set for each event
Instead of specifying a
levelto be used for all events sent to Rollbar, you now specify adefault_level.You can override this level by adding a field to the event like:
Note that if the value passed to Rollbar for level isn't recognized (i.e. isn't one of
debug,info,warning,error, orcritical) Rollbar will consider call it an Error even if your default level is something other than Error.