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

Add Certificate Bundle Support To WebSocket Client (IDFGH-9404) #224

Closed
drguthrie opened this issue Feb 15, 2023 · 3 comments
Closed

Add Certificate Bundle Support To WebSocket Client (IDFGH-9404) #224

drguthrie opened this issue Feb 15, 2023 · 3 comments

Comments

@drguthrie
Copy link

Hello,

HTTP Client has support for the certificate bundle but WebsSocket Client does not. Is there any interest in adding support for the certificate bundle into WebSocket Client? I got it to work with only a few changes by following the code in the HTTP Client initialization method.

If there is interest I would be happy to make the changes and submit a pull request.

@drguthrie drguthrie changed the title Add Certificate Bundle Support Add Certificate Bundle Support To WebSocket Client Feb 15, 2023
@github-actions github-actions bot changed the title Add Certificate Bundle Support To WebSocket Client Add Certificate Bundle Support To WebSocket Client (IDFGH-9404) Feb 15, 2023
@david-cermak
Copy link
Collaborator

Hello @drguthrie

We welcome all contributions and the certificate bundle is a very useful addition to the websocket client.
Please check the contribution guidelines and post a PR with your changes. Thanks for the help!

@toastedcrumpets
Copy link

Hi @drguthrie,
Can you offer your patch code for review? I have the same problem. I will look at patching too but would like to collaborate on an update for ESP IDF.
Marcus

@david-cermak
Copy link
Collaborator

It seems to me that the certificate bundle is already supported:

esp_err_t (*crt_bundle_attach)(void *conf); /*!< Function pointer to esp_crt_bundle_attach. Enables the use of certification bundle for server verification, MBEDTLS_CERTIFICATE_BUNDLE must be enabled in menuconfig. Include esp_crt_bundle.h, and use `esp_crt_bundle_attach` here to include bundled CA certificates. */

Implemented in d56b5d9

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

No branches or pull requests

4 participants