Skip to content
This repository has been archived by the owner on Aug 30, 2023. It is now read-only.

Proxy support #14

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Proxy support #14

wants to merge 1 commit into from

Conversation

themylogin
Copy link

As aiohttp won't support setting up proxy from environment variables (aio-libs/aiohttp#529), I've had to implement it as a transport option.

Example usage:

sentry_dsn = "https://public_key:private_key@sentry.io/1234?proxy=http%3A%2F%2Fproxy%3A3128"

@asvetlov
Copy link

See also aio-libs/aiohttp#2256 -- it introduces proxy_from_env() function.

@ashwoods
Copy link
Contributor

And afaik Sentry deprecated options on the DSN, at least for the transport, in favor of explicit class loading: getsentry/raven-python#587

@azhurbilo
Copy link

What the status of this Proxy feature?
May be others know any workaround?

It's impossible to use this client when application access to public internet blocked by firewall :(

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants