Skip to content
This repository has been archived by the owner on Sep 29, 2024. It is now read-only.

TLS negotiation times out with ProtonVPN #230

Closed
keeshux opened this issue Nov 19, 2021 · 1 comment · Fixed by #233
Closed

TLS negotiation times out with ProtonVPN #230

keeshux opened this issue Nov 19, 2021 · 1 comment · Fixed by #233
Assignees
Labels
bug Something isn't working help wanted Extra attention is needed regression Used to work but doesn't anymore
Milestone

Comments

@keeshux
Copy link
Member

keeshux commented Nov 19, 2021

Started with BoringSSL in TunnelKit.

@keeshux keeshux self-assigned this Nov 19, 2021
@keeshux
Copy link
Member Author

keeshux commented Nov 23, 2021

From a client perspective, behavior is equal to e.g. when server expects a client certificate and client doesn't specify one. In that case, server stops responding from the very same point, i.e. when TLS handshake starts (right after HARD_RESET).

This is what happens on the server in that scenario:

2021-11-22 09:31:27 us=7187 90.166.54.153:23086 OpenSSL: error:1417C0C7:SSL routines:tls_process_client_certificate:peer did not return a certificate
2021-11-22 09:31:27 us=7328 90.166.54.153:23086 TLS_ERROR: BIO read tls_read_plaintext error
2021-11-22 09:31:27 us=7466 90.166.54.153:23086 TLS Error: TLS object -> incoming plaintext read error
2021-11-22 09:31:27 us=7581 90.166.54.153:23086 TLS Error: TLS handshake failed
2021-11-22 09:31:27 us=7781 90.166.54.153:23086 SIGUSR1[soft,tls-error] received, client-instance restarting

Worth noting that NordVPN, like ProtonVPN, also has a tls-auth option w/o client certificates. Nevertheless, NordVPN doesn't have this issue. Tested a similar configuration successfully on personal server too.

All in all, this used to work with an OpenSSL client. TLS may fail for plenty of reasons unknown without looking at the server logs.

@keeshux keeshux transferred this issue from passepartoutvpn/passepartout Nov 23, 2021
@keeshux keeshux added bug Something isn't working help wanted Extra attention is needed regression Used to work but doesn't anymore labels Nov 23, 2021
@keeshux keeshux modified the milestones: 4.1.0, 4.0.2 Nov 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working help wanted Extra attention is needed regression Used to work but doesn't anymore
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant