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

Implement async certificate verification for the managed buffers API #8

Open
pvdrz opened this issue Nov 2, 2023 · 0 comments
Open

Implement async certificate verification for the managed buffers API #8

pvdrz opened this issue Nov 2, 2023 · 0 comments
Labels
nice to have It is not clear if this is strictly necessary to submit a PR upstream.

Comments

@pvdrz
Copy link
Collaborator

pvdrz commented Nov 2, 2023

The logic in src/low_level.rs does not support async certificate verification. This part of the API is described in Jorge's RFC and needs to be implemented. Right now, the certificate verification is done in src/client/low_level.rs by calling verify_server_cert inside ExpectCertificate.

It is not clear if this is strictly necessary to submit a PR upstream.
@pvdrz pvdrz added the nice to have It is not clear if this is strictly necessary to submit a PR upstream. label Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
nice to have It is not clear if this is strictly necessary to submit a PR upstream.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@pvdrz