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

bearssl: Use common code for cipher suite lookup #13464

Closed
wants to merge 1 commit into from
Closed

bearssl: Use common code for cipher suite lookup #13464

wants to merge 1 commit into from

Conversation

jan2000
Copy link
Contributor

@jan2000 jan2000 commented Apr 24, 2024

Take advantage of the Curl_cipher_suite_walk_str() and Curl_cipher_suite_get_str() functions introduced in commit fba9afe.

This also fixes CURLOPT_SSL_CIPHER_LIST not working at all for bearssl due to commit ff74cef.

@jan2000
bearssl: Use common code for cipher suite lookup
52ed568 
Take advantage of the Curl_cipher_suite_walk_str() and
Curl_cipher_suite_get_str() functions introduced in
commit fba9afe.

This also fixes CURLOPT_SSL_CIPHER_LIST not working at all for
bearssl due to commit ff74cef.
@github-actions github-actions bot added the tests label Apr 24, 2024
@bagder bagder added the TLS label May 2, 2024
@bagder bagder closed this in 6e4b7ab May 2, 2024
@bagder
Copy link
Member

bagder commented May 2, 2024

Thanks!

@jan2000 jan2000 deleted the bearssl-update-ssl-cipher-list branch May 4, 2024 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bagder bagder bagder approved these changes

Assignees
No one assigned
Labels
tests TLS
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jan2000 @bagder