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.

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

Process only ERROR level #41

Closed
jglapa opened this issue Sep 22, 2017 · 3 comments
Closed

Process only ERROR level #41

jglapa opened this issue Sep 22, 2017 · 3 comments
Labels

Comments

@jglapa
Copy link

jglapa commented Sep 22, 2017

How to configure the plugin to login only ERROR events?

Having this in my config I would expect only error events to be published

grails.plugin.sentry.levels = ['ERROR']

but looking at the source code it will fall back to the defaults which contains also WARN

if (config.levels && config.levels.contains(event.level)) {
super.append(event)
} else if (SentryConfig.defaultLevels.contains(event.level)) {
super.append(event)
}

@musketyr musketyr added the bug label Sep 26, 2017
@musketyr
Copy link
Collaborator

Well spotted. The condition should be changed.

@jglapa
Copy link
Author

jglapa commented Sep 26, 2017

do you accept PRs?

@musketyr
Copy link
Collaborator

PRs are always welcomed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants