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.

Sign up for GitHub

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

Jump to bottom

Documentation claims SNI is supported, however it's not. (IDFGH-8340) #9815

Closed
3 tasks done
axos88 opened this issue Sep 19, 2022 · 0 comments
Closed
3 tasks done

Documentation claims SNI is supported, however it's not. (IDFGH-8340) #9815

axos88 opened this issue Sep 19, 2022 · 0 comments
Assignees
@AdityaHPatwardhan
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally

Comments

@axos88
Copy link
Contributor

axos88 commented Sep 19, 2022

Answers checklist.

  • I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
  • I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
  • I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report

Looking through the source code, at least in case of mbedtls, there is no support for server-side SNI.
A call to mbedtls_ssl_conf_sni is not found in the code that would allow the caller to hook into SNI and for example choose a server certificate based on the requested hostname of the client.

https://mbed-tls.readthedocs.io/en/latest/kb/how-to/use-sni/
@espressif-bot espressif-bot added the Status: Opened Issue is new label Sep 19, 2022
@github-actions github-actions bot changed the title Documentation claims SNI is supported, however it's not. Documentation claims SNI is supported, however it's not. (IDFGH-8340) Sep 19, 2022
@espressif-bot espressif-bot added Status: In Progress Work is in progress and removed Status: Opened Issue is new labels Sep 27, 2022
@espressif-bot espressif-bot added Resolution: Done Issue is done internally Status: Done Issue is done internally and removed Status: In Progress Work is in progress labels Nov 4, 2022
chipweinberger pushed a commit to chipweinberger/esp-idf that referenced this issue Nov 8, 2022
@mahavirj @chipweinberger
Merge branch 'feature/esp_tls_add_cert_selection_callback' into 'master'
2c79bf9 
esp-tls: Add support for the CERTIFICATE SELECTION HOOK. The hook has access...

Closes IDFGH-8363 and IDFGH-8340
Closes espressif#9815

See merge request espressif/esp-idf!20690
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AdityaHPatwardhan AdityaHPatwardhan

Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@mahavirj @axos88 @AdityaHPatwardhan @espressif-bot