RSDK-5399 - surface component errors in gRPC handler

[gvaradarajan]

No description provided.

[npmenard]

I would like to avoid using error code not officially listed by gRPC (see https://grpc.github.io/grpc/core/md_doc_statuscodes.html)
We can extend the GrpcError type to allow passing a cause and do something like err.to_string()

[npmenard]

Could we use thiserror like
#[derive(Error, Debug)] pub struct MyError { grpc_error: GrpcError, #[source] // optional if field name issourcesource: Option<anyhow::Error>, }

Or is it causing issue with non standard fields?

[gvaradarajan]

I don't quite get what you mean here? Are you saying this?

#[derive(Debug, Error)]
pub struct ServerError {
    grpc_error: GrpcError,
    #[source]
    cause: Option<anyhow::Error>,
}

The reason I didn't do that was because if a cause wasn't provided I wanted the source to be the GrpcError

[gvaradarajan]

Never mind, I don't even know why I wanted that I'll change it

[npmenard]

LGTM

[npmenard]

you may be able to delegate to Error's Display implementation

[gvaradarajan]

it was a lot more complicated to try to do this because the Debug and Display traits both have the fmt function with the same signature