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

Track rustls 0.22 API changes #320

Closed
wants to merge 4 commits into from
Closed

Conversation

ctz
Copy link
Member

@ctz ctz commented Jun 29, 2023

Just parking this here.

@cpu
Copy link
Member

cpu commented Jun 29, 2023

fwiw I've also started on integrating rustls/rustls#1326

I'm going to try and land an update to client.c and server.c to add mTLS support w/ the existing API separately before I muddy the waters with the changes to pick up the CRL support aspect.

@cpu
Copy link
Member

cpu commented Jun 29, 2023

I'm working through some of the main branch CI failures. I'll try to get a PR up shortly and we can rebase both this + #321 and hopefully get back to green CI 🤞

@ctz
Copy link
Member Author

ctz commented Jun 29, 2023

I'm working through some of the main branch CI failures.

Excellent! I think maybe we should add a periodic build each night to flush out these earlier.

@cpu
Copy link
Member

cpu commented Jun 29, 2023

Excellent! I think maybe we should add a periodic build each night to flush out these earlier.

+1 - I can handle that too.

@ctz
Copy link
Member Author

ctz commented Jul 10, 2023

Going to close this -- as it happened 0.22 wasn't a breaking build, and this stuff for 0.23 can come later.

@ctz ctz closed this Jul 10, 2023
@cpu cpu deleted the jbp-track-rustls-0.22-changes branch December 21, 2024 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants