Skip to content

Conversation

sreimers
Copy link
Member

No description provided.

@sreimers sreimers force-pushed the openssl_libressl_update branch from 3df284a to a012beb Compare May 22, 2024 08:54
@alfredh
Copy link
Contributor

alfredh commented Jun 1, 2024

is there a problem with the LibreSSL build ?

reqconn: Could not send GET request. (Function not implemented [38])

@sreimers
Copy link
Member Author

sreimers commented Jun 1, 2024

Yes, looks like some SSL functions are not implemented for libressl and retest fails with ENOSYS.

@sreimers sreimers force-pushed the openssl_libressl_update branch from a012beb to 7057dc7 Compare June 6, 2024 13:36
@sreimers sreimers force-pushed the openssl_libressl_update branch 2 times, most recently from 54abf40 to e8be4b7 Compare May 12, 2025 14:46
@alfredh
Copy link
Contributor

alfredh commented May 12, 2025

I think ssl_err=2 means SSL_ERR_WANT_READ

https://github.com/openssl/openssl/blob/master/include/openssl/ssl.h.in#L1206

This means that the operation has to be repeated ...
similar logic is done in tls_tcp.c

@sreimers
Copy link
Member Author

tls: SSL_verify_client_post_handshake error: Bad address [14], ssl_err=2

LibreSSL only stubs the SSL_verify_client_post_handshake API and returns always a error if used.

TLS 1.3 post handshakes are an extension, and HTTP/2, for example, explicitly forbids them (because of the multiplexing). So it's a very rarely used feature. We should add a feature/flag detection for this, I think.

@sreimers sreimers force-pushed the openssl_libressl_update branch 2 times, most recently from 93d2393 to 06f36d8 Compare May 13, 2025 15:29
@sreimers sreimers force-pushed the openssl_libressl_update branch from 06f36d8 to 4d9be2c Compare July 29, 2025 07:36
@sreimers sreimers merged commit 957116b into main Jul 29, 2025
38 checks passed
@sreimers sreimers deleted the openssl_libressl_update branch July 29, 2025 08:11
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.

2 participants