Sija/raven.cr

https

crisward opened this issue · 8 comments

Has this been tested with https ?
My sentry server has ssl, and I'm getting the following error from the failure_callback -

ERROR -- sentry: Unable to record event with remote Sentry server (Raven::Error - Moved Permanently): 0x944245: ??? at /app/lib/raven/src/raven/transports/http.cr 64:11

I'm pretty sure this happens when traffic is redirected from http, to https? Just a guess though as I've not read the source.

Sija commented

Stupid question, but did url in SENTRY_DSN use https scheme?
If not, more logs captured with config.logger.level = Logger::DEBUG might help.

It's not a stupid question, but yes I used https:// in my DSN. I'll try adding logger.level.

Sija commented

I think I've found the culprit. Could you please try develop branch and see does it work?

Will do, thanks.

Thanks. That worked.

Also, with my test error I got this. I'm using sentry 8.05 (I installed in a while ago, possibly due an update)

screen shot 2017-03-20 at 08 39 21

Sija commented

gr8! I'm closing this issue and will release patched version soon, in the meantime you can stick to develop branch.

Sija commented

Regarding the errors, they're caused by the fields added l8r on to the Sentry server API. These error messages are harmless, yet you might miss OS information section (See https://docs.sentry.io/clientdev/interfaces/contexts/).