[PR] Add request URI to gRPC client error descriptions [pekko-grpc]

["pjfanning (via GitHub)" <gi...@apache.org>]

pjfanning opened a new pull request, #646:
URL: https://github.com/apache/pekko-grpc/pull/646

   Part of #642 
   
   Copies https://github.com/akka/akka-grpc/pull/1748 which is now Apache licensed


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org

Re: [PR] Add request URI to gRPC client error descriptions [pekko-grpc]

["pjfanning (via GitHub)" <gi...@apache.org>]

pjfanning commented on PR #646:
URL: https://github.com/apache/pekko-grpc/pull/646#issuecomment-4012476810

   @raboof reasonable point about URLs in exception messages.
   
   We could make the behaviour opt-in via config (off by default).
   Or just log the URLs to local logs instead of adding them to exception messages (if this is not already done)
   
   Noone has explicitly asked for this and we could just take this as a cue to leave this for now.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org