Support SOCKS proxies #1049

Activity

1. Roguelazer

   added 3 commits that reference this issue on Mar 10, 2021

   support SOCKS proxies in sentry_sdk

   ...

   This issue will close once commit 556a32d is merged into the 'master' branch.

   556a32d

   support SOCKS proxies in sentry_sdk

   ...

   This issue will close once commit 8493d96 is merged into the 'master' branch.

   8493d96

   support SOCKS proxies in sentry_sdk

   ...

   This issue will close once commit f60662e is merged into the 'master' branch.

   f60662e
2. kenoss commented on Jun 17, 2021

   

   kenoss

   on Jun 17, 2021

   More actions

   I vote one. It is great that sentry_sdk itself has SOCKS proxy support (integrated in the class HttpTransport.).

   Rationale:

   1. There are situations we have to use sentry under SOCKS proxy. E.g., IoT device with restricted network access.
   2. I tried to write external class Socks5HttpTransport, but it's not cool to use.

   And then, here's some implementation options.

   How to designate proxy URL

   A. From http_proxy/https_proxy option, like requests and #1050 .
   B. From other new option like socks_proxy.

   I guess the members of sentry prefer B because #178 (comment) .

   ---

   @untitaker @ahmedetefy Could you let us know your thoughts?
3. sl0thentr0py

   self-assigned this

   on Dec 20, 2021
4. sl0thentr0py

   added

   enhancement

   Status: Backlog

   on Dec 20, 2021
5. antonpirker

   closed this as completedin #1050on Jun 19, 2023