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

Is https not supported? #794

Open
yuting-fan opened this issue Jun 22, 2022 · 3 comments · May be fixed by #916
Open

Is https not supported? #794

yuting-fan opened this issue Jun 22, 2022 · 3 comments · May be fixed by #916
Labels

Comments

@yuting-fan
Copy link

It looks like gorilla websocket intends to support https, e.g logic here.

However, https is not registered during init here.

This caused https to be considered as an unknown proxy when we use https proxy, error thrown from here.

Is this a bug? Shouldn't the proxy_RegisterDialerType be initialized with https?

@ayjayt
Copy link

ayjayt commented Jul 21, 2023

Websockets does support https. The client dialers expect you to pass scheme wss://, not https://.

That being said, the behavior your mentioning IMO is an issue (so picky about scheme), but bottom line is that gorilla websocket does support TLS and will connect to an https server.

@ayjayt
Copy link

ayjayt commented Jul 21, 2023

Infuriating:

switch u.Scheme {

@ghost
Copy link

ghost commented Jul 21, 2023

The issue is asking about support for HTTPS proxy servers, not whether the package supports WSS or not.

To reduce confusion, the issue title should be changed to "Are HTTPS proxy servers supported?"

@jackstar12 jackstar12 linked a pull request May 9, 2024 that will close this issue
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: No status
Development

Successfully merging a pull request may close this issue.

4 participants