You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I've searched the issue tracker for similar requests
Is your feature request related to a problem? Please describe.
I'm doing some complex TLS certificate resolving logic and the fact that ResolvesServerCert trait takes ClientHello by value rather than by reference is a bit annoying. Since it also does not implement Clone and cannot be created (new is private to crate) I cannot pass it to more than one other implementer of ResolvesServerCert and that's sometimes needed.
I understand that it's minor issue and would break API, but maybe in some future versions it can be improved.
Thanks!
Describe the solution you'd like
Pass ClientHello by reference / make it Clone / wrap in Arc
The text was updated successfully, but these errors were encountered:
@djc Yes, thanks, I know about it, but for me ResolvesServerCert is mostly enough. I solved my case by adding my own internal trait that takes ClientHello by ref, so at least for now it's not a problem.
Just an idea for future releases that it might make people's life a bit easier.
Checklist
Is your feature request related to a problem? Please describe.
I'm doing some complex TLS certificate resolving logic and the fact that
ResolvesServerCert
trait takesClientHello
by value rather than by reference is a bit annoying. Since it also does not implementClone
and cannot be created (new
is private to crate) I cannot pass it to more than one other implementer ofResolvesServerCert
and that's sometimes needed.I understand that it's minor issue and would break API, but maybe in some future versions it can be improved.
Thanks!
Describe the solution you'd like
Pass
ClientHello
by reference / make itClone
/ wrap inArc
The text was updated successfully, but these errors were encountered: