Trustin Lee

Results 271 comments of Trustin Lee

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