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 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

@bagder
Copy link
Member

@bagder 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 approved these changes Mar 10, 2022
@bagder
Copy link
Member

@bagder bagder commented Mar 12, 2022

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants