-
Notifications
You must be signed in to change notification settings - Fork 434
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
1.22.0 causes CLIENT ALERT: Fatal - Handshake Failure
when sending GET request with TLS options
#759
Comments
Due to an error on Hackney 1.22.0 when sending TLS requests, limit it to the latest compatible version. More info: benoitc/hackney#759
Due to an error on Hackney 1.22.0 when sending TLS requests, limit it to the latest compatible version. More info: benoitc/hackney#759
can you check with #760 this should fix your issue. |
@benoitc the issue is gone. Thank you very much! |
It is still happening after 1.23.0, the difference is that this time it fails when the request is a |
there is no difference between POST and GET on connect normally. can you share a trace of your issue? |
@luismiramirez can you try to passe the option |
Sure! Here's the trace. No handshake error anymore, now it is a connection error:
|
My bad it should be |
Sure, here's the handshake error again:
|
this is because the host enforse domain name verifictaion. I am looking at it. |
@luismiramirez please check #763 and let me know. This should fix the issue you reported. |
@luismiramirez were you able to check? |
@benoitc yes, I pasted the trace in #763
|
@luismiramirez sorry I missed that. checking |
I have same issue with 1.23.0 and I cannot replicate with 1.20.1 |
@silviucpp can you share the url you're trying. I don't reproduce when #763 is applied . |
Description
When calling
:hackney.request/4
with TLS options including custom hostname verification as described in the trace below, we're receiving handshake failures. This is a regression as we've confirmed this works correctly in Hackney 1.21.0.Steps to reproduce
Trace
The text was updated successfully, but these errors were encountered: