hyperium / tonic

A native gRPC client & server implementation with async/await support.
https://docs.rs/tonic
MIT License
9.35k stars 957 forks source link

Expose Error Status To Caller #1682

Closed zhixinwen closed 2 weeks ago

zhixinwen commented 2 months ago

Motivation

We were seeing a high rate Missing request message error on server side, but we do not know what exactly was going on. I think the error message may be hidden away in poll_data. I am not sure why tonic choose to drop the error message instead of returning it to the user. Can people familiar with the code explain why it was done in the current way?

Solution

return the error to users, so the error is more meaningful