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
ClientConfig's transport has pub(crate), while ServerConfig's transport has pub. Is this difference intentional? I think both should be pub so that updating the client's transport is easier (unless I'm not supposed to be doing that? I'm new to this).
The text was updated successfully, but these errors were encountered:
BlinkyStitt
added a commit
to BlinkyStitt/quic-tunnel
that referenced
this issue
Dec 3, 2023
The intention is that you should generally use the setter methods. Does that work for your case? Perhaps we should privatize the ServerConfig fields for consistency.
Every time I try to clean this up it turns out to be more complicated than I expected, but we should definitely try to be consistent one way or another here.
ClientConfig's transport has
pub(crate)
, while ServerConfig's transport haspub
. Is this difference intentional? I think both should be pub so that updating the client's transport is easier (unless I'm not supposed to be doing that? I'm new to this).The text was updated successfully, but these errors were encountered: