Skip to content

Comments

Add proxy support.#146

Open
thibwk wants to merge 1 commit intosignalkraft:mainfrom
thibwk:main
Open

Add proxy support.#146
thibwk wants to merge 1 commit intosignalkraft:mainfrom
thibwk:main

Conversation

@thibwk
Copy link

@thibwk thibwk commented Dec 14, 2025

aiohttp doesn't trust the environment by default, so it doesn't use HTTPS_PROXY.

This will allow for a proxy to be set explicitly when instantiating the API.

@signalkraft
Copy link
Owner

Thanks for the PR, and sorry for the late response here.

I think it might best to extend the API __init__ with **kwargs and pass those along to the ClientSession, to support whatever other aiohttp features people may want to use. For example there's trust_env=True to read proxy settings from env vars.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants