feat: add new method get_uri for Endpoint #371
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds a new method
get_uri
forEndpoint
, to allow user to get the&Uri
from anEndpoint
Motivation
I am using
tonic
and stored theEndpoint
, ensure I can reconnect GRPC server directly, however sometimes I need to get the uri inEndpoint
to print some log.One way is to create a wrapper
struct Endpoint(Uri, tonic::transport::Endpoint)
, but it will allocate a newUri
.I think the better way is let
Endpoint
can return theUri
reference which is in theEndpoint
, user can use it to print logs or do something else, without unnecessary allocating.related issue is #370