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

coap-mbedtls.c: Support 3.6.0 Mbed TLS #1458

Merged
merged 1 commit into from
Jul 11, 2024
Merged

Conversation

mrdeep1
Copy link
Collaborator

@mrdeep1 mrdeep1 commented Jul 4, 2024

TLS1.3 is enabled by default in 3.6.0. This PR fixes the supported encryption algorithms as well as handling a new session ticket indication for coaps+tcp://.

However, if client certificates are not defined, then MbedTLS 3.6.0 fails with no certificate CA, even if the CA Cert is presented along with the Server Cert during the processing of the server certificate and fails with "SSL - No CA Chain is set, but required to operate" when coaps+tcp:// is used. See Mbed-TLS/mbedtls#9223

Workaround (with examples/coap-client) is to set the -R options (when the -c option is not set), or to programmatically call coap_context_set_pki_root_cas);, or to use a MBedTLS version < 3.6.0.

TLS1.3 is enabled by default in 3.6.0. This PR fixes the supported
encryption algorithms as well as handling a new session ticket
indication for coaps+tcp://.

However, if client certificates are not defined, then MbedTLS fails with
no certificate CA, even if the CA Cert is presented along with the Server
Cert during the processing of the server certificate and fails with
"SSL - No CA Chain is set, but required to operate"
when coaps+tcp:// is used.

Workaround (with examples/coap-client) is to set the -R options (when the
-c option is not set), or to programmatically call
coap_context_set_pki_root_cas();
@mrdeep1 mrdeep1 merged commit 7d5e5f3 into obgm:develop Jul 11, 2024
31 checks passed
@mrdeep1 mrdeep1 deleted the mbedtls_3_6_0 branch July 11, 2024 11:53
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.

1 participant