Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support Corporate Proxy #74

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

potofo
Copy link

@potofo potofo commented Jan 25, 2025

Thank you for the wonderful sample program.
I tested it both at home and at my company. In the company environment, Python’s WebSocket couldn’t bypass the corporate proxy.
So, I added the aiohttp_proxy package to follow the HTTPS_PROXY environment variable and allow it to bypass the corporate proxy. I have confirmed that if the HTTPS_PROXY variable is not set, it will attempt a direct connection.

I hope that many engineers testing this sample program in a corporate environment can use it to learn about the Realtime API.
Please accept my pull request and reflect these changes in the main repository.

@potofo
Copy link
Author

potofo commented Jan 25, 2025

@microsoft-github-policy-service agree

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.

1 participant