agorapulse / grails-sentry

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

`unknown` value for ipAddress is unable to be parsed by sentry.io #45

Closed erichelgeson closed 6 years ago

erichelgeson commented 6 years ago

I recently forgot to turn on request logging (so the request obj was not available to parse) and the plugin tried to use the value unknown for the ipAddress of the sentry.interfaces.User. sentry.io was unable to parse this -

screen shot 2018-01-25 at 1 05 16 pm

A work around may be as simple as having the default be 127.0.0.1 instead of unknown. Let me know (I can open a PR if that's an acceptable work around)

musketyr commented 6 years ago

Sounds good, you can definitely open a PR. Thanks