Option to set "transport" in the config

nameko / nameko-sentry

Apache License 2.0

28 stars 9 forks source link

Option to set "transport" in the config #19

Closed deepinder19 closed 5 years ago

deepinder19 commented 5 years ago

transport has been hardcoded to EventletHTTPTransport but it would be nice if we can have the option to set it in the config file.

deepinder19 commented 5 years ago

The issue is that I am able to send exceptions to sentry using ThreadedHTTPTransport but not using EventletHTTPTransport.

mattbennett commented 5 years ago

@deepinder19 why doesn't EventletHTTPTransport work for you?

Nameko relies on Eventlet, so nameko-sentry using eventlet threads should be fine...

deepinder19 commented 5 years ago

The issue was related to ssl certificates error on my local machine. Apparently ssl certificates needs to be installed manually if using Python 3.6 on Mac. And sentry was failing silently because of this error.