search

agorapulse / grails-sentry

A Sentry client for Grails
Apache License 2.0
12 stars 14 forks source link

Fixed boolean config #48

Closed donbeave closed 6 years ago

donbeave commented 6 years ago

Hi!

It's a small fix. Only for apps which use yaml as a config. Sometimes after parsing yaml some values will be transformed to String instead of Boolean. In this case, SentryConfig doesn't work correctly. This fix just compares all values as a String.

donbeave commented 6 years ago

Sorry for disturbing you. @musketyr, do you have time to take a look on this PR?

musketyr commented 6 years ago

no worries, I have missed this one. thanks!

donbeave commented 6 years ago

Thank you!