Merge pull request #1420 from murgatroid99/grpc-js_serialization_error_details

grpc-js: Add more detailed information to client (de)serialization errors
This commit is contained in:
Michael Lumish 2020-05-21 10:07:14 -07:00 committed by GitHub
commit ca4cb322a2
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions

View File

@ -359,7 +359,7 @@ class BaseInterceptingCall implements InterceptingCallInterface {
serialized = this.methodDefinition.requestSerialize(message);
this.call.sendMessageWithContext(context, serialized);
} catch (e) {
this.call.cancelWithStatus(Status.INTERNAL, 'Serialization failure');
this.call.cancelWithStatus(Status.INTERNAL, `Request message serialization failure: ${e.message}`);
}
}
// eslint-disable-next-line @typescript-eslint/no-explicit-any
@ -384,7 +384,7 @@ class BaseInterceptingCall implements InterceptingCallInterface {
} catch (e) {
readError = {
code: Status.INTERNAL,
details: 'Failed to parse server response',
details: `Response message parsing error: ${e.message}`,
metadata: new Metadata(),
};
this.call.cancelWithStatus(readError.code, readError.details);