Trustin Lee
Trustin Lee
Gentle ping :pray:
Thanks, @rsmckinney!
I'm not sure renaming is a scalable approach. What happens if we add support for other RPC protocols, say JSON-RPC, Hessian, RSocket etc? Why not just make `RpcRequest/Response` a `StreamMessage`...
As a stopgap solution, how about checking if RPC decorators are present in pure HTTP clients and raising an exception?
I guess you're going to update this PR again once #5610 is merged, right, @jrhee17?
Both are fine by me. Let me know if you need this PR to be merged, we'd be happy to accept it given it will take quite some time until...
I think we should go with Micrometer Observation as described at #5736. Sorry but let me close this. :bow:
@minwoox, @Kyurenpoto I think we should go with Micrometer Observation as described at #5736. Sorry but let me close this. :bow:
Instead, we could consider encrypting the entire repository? https://git-annex.branchable.com/tips/fully_encrypted_git_repositories_with_gcrypt/
We might want to wait for `Micrometer Tracing` integration in Armeria first though