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.

Sign up for GitHub

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

Jump to bottom

Incorrect default port for wss #21

Open
edubart opened this issue Jun 3, 2017 · 2 comments
Open

Incorrect default port for wss #21

edubart opened this issue Jun 3, 2017 · 2 comments

Comments

@edubart
Copy link

edubart commented Jun 3, 2017

The library is using port 80 instead of 443 for wss:// URLs, giving the following errors:

2017/06/03 13:20:12 [error] 5776#0: *2 SSL_do_handshake() failed (SSL: error:140770FC:SSL routines:SSL23_GET_
SERVER_HELLO:unknown protocol), context: ngx.timer

and

2017/06/03 13:10:52 [error] 3441#0: *2 peer closed connection in SSL handshake, context: ngx.timer
ssl handshake failed: handshake failed

Using :443 in the URL fixes the problem, like wss://echo.websocket.org:443
@agentzh
Copy link
Member

agentzh commented Jun 3, 2017

@edubart Hmm, will you mind creating a pull request to fix this issue? Many thanks!

@edubart edubart mentioned this issue Jun 5, 2017
Closed
@lilien1010 lilien1010 mentioned this issue Jul 14, 2017
Closed
jprjr added a commit to jprjr/lua-resty-websocket that referenced this issue Dec 24, 2020
@jprjr
client: default to port 443 for wss urls
a56f08d 
fixes issue openresty#21
This was referenced Dec 24, 2020
Merged
Merged
@jprjr
Copy link
Contributor

jprjr commented Dec 24, 2020

Hi all, opened pr #55 to address this.

zhuizhuhaomeng pushed a commit that referenced this issue Mar 9, 2022
@jprjr
bugfix: default to port 443 for wss urls (#55)
697acc0 
fixes issue #21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use default port 443 for wss:// edubart/lua-resty-websocket
3 participants
@agentzh @edubart @jprjr