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

SSL, some tidy ups #12204

Closed
wants to merge 4 commits into from
Closed

SSL, some tidy ups #12204

wants to merge 4 commits into from

Conversation

icing
Copy link
Contributor

@icing icing commented Oct 26, 2023

Move all handling of connection SSL config into vtls.c

  • init defaults, cleanup
  • inheriting from easy handle
  • updating when easy handle is changed
  • use Curl_ssl_cf_get_primary_config() in QUIC filters
  • remove Curl_ssl_get_config(), no longer needed

bagder
bagder approved these changes Oct 26, 2023
View reviewed changes
Copy link
Member

@bagder bagder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Me like!

@icing
vtls, remove Curl_ssl_get_config(), no longer needed
1277b11
@icing
move all handling of connection SSL config into vtls.c
7df899c 
- init defaults, cleanup
- inheriting from easy handle
- updating when easy handle is changed
- use Curl_ssl_cf_get_primary_config() in QUIC filters
@icing
ngtcp2+wolfssl, fix compile error
ec60430
@icing
url.c, handle failed Curl_ssl_conn_config_init()
794e5fa
@icing icing requested a review from bagder October 30, 2023 19:33
@bagder bagder closed this in bf0e278 Oct 31, 2023
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
hacktoberfest-accepted tidy-up TLS
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@icing @bagder