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

Implemented BearSSL: CURLOPT_SSL_CIPHER_LIST #8477

Closed
wants to merge 4 commits into from

Conversation

jan2000
Copy link
Contributor

@jan2000 jan2000 commented Feb 19, 2022

Separated from: #8106

Implemented the CURLOPT_SSL_CIPHER_LIST option for BearSSL based on the code of SecureTransport.

@jay
Copy link
Member

jay commented Feb 19, 2022

You'll have to update the documentation as well. There's CIPHERS.md, CURLOPT_SSL_CIPHER_LIST.3, CURLOPT_PROXY_SSL_CIPHER_LIST.3. Also there's TLS13_cipher docs that need to be updated if you added support for TLS 1.3 ciphers

@jay jay added TLS feature-window A merge of this requires an open feature window labels Feb 19, 2022
@bagder
Copy link
Member

bagder commented Mar 9, 2022

BearSSL doesn't support TLS 1.3 at all. Which makes me wonder why people are even using this library at it seems rather limited. It also seems to have been abandoned, as there's been no commits to their git repo in over a year.

@bagder bagder removed the feature-window A merge of this requires an open feature window label Mar 10, 2022
@bagder
Copy link
Member

bagder commented Mar 12, 2022

Thanks!

@bagder bagder closed this in f5d7961 Mar 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

3 participants