Daniel McCarney
Daniel McCarney
> If we removed the RecursorPool encapsulation, we could separately store connections in one cache, indexed by IP and protocol, and zone name server pool information in another cache, indexed...
@divergentdave Do you have thoughts on this & #3210 as the OP of this issue?
I believe the Deno folks have worked on something in this space: https://github.com/denoland/rustls-tokio-stream that might be a useful comparison point. There's some previous discussion in an issue that was created...
> @cpu tokio-rustls is not affected by https://github.com/tokio-rs/tls/issues/40 . Ah! Thanks for clarifying. I had indeed missed that this discussion was centered around native-tls.
@gendx Was it intentional to put the test vectors in the schema dir? I think we'd probably want them in `testvectors_v1` (?) or another top-level tests directory.
@FiloSottile Do you want to take a look at these before merging?
@bwesterb I saw you ref'd these vectors in https://boringssl-review.googlesource.com/c/boringssl/+/79947 Would you be interested in taking a look at the updated branch?
> reviewing these test vectors independently That's what I was thinking. Taking a look at the branch, commenting that the vectors worked for your needs previously, and hopefully that the...
> I've https://github.com/cloudflare/circl/pull/552 against our implementation in Go. These look great. Awesome, thanks for taking a look. I'm going to merge this PR now on the back of my review...
Thanks again gendx !