Skip to content

Trailing metadata is the entire Status proto, not just error details

Jeremiah Bonney requested to merge jbonney/fix-status-details2 into master

Description

Third time is the charm for getting RetryInfo support.

My previous MR (!680 (merged)) started setting the RetryInfo message as part of grpc-status-details-bin trailing metadata, however that was the wrong thing to do. The value of this key is supposed to be the entire Status message, as opposed to just the details section. I've gone ahead and updated the MR/test, and added some log lines when we handle this exception to get more visibility into when we are actually returning this to clients.

Merge request reports