Skip to content
Snippets Groups Projects
Commit 655e28f8 authored by Jan Tattermusch's avatar Jan Tattermusch Committed by GitHub
Browse files

Merge pull request #6904 from nathanielmanistaatgoogle/unimplemented-for-cardinality-violation

UNIMPLEMENTED status for cardinality violation
parents 6f42fd92 04422141
Branches
Tags
No related merge requests found
...@@ -344,10 +344,9 @@ def _unary_request(rpc_event, state, request_deserializer): ...@@ -344,10 +344,9 @@ def _unary_request(rpc_event, state, request_deserializer):
if state.client is _CLOSED: if state.client is _CLOSED:
details = '"{}" requires exactly one request message.'.format( details = '"{}" requires exactly one request message.'.format(
rpc_event.request_call_details.method) rpc_event.request_call_details.method)
# TODO(5992#issuecomment-220761992): really, what status code?
_abort( _abort(
state, rpc_event.operation_call, state, rpc_event.operation_call,
cygrpc.StatusCode.unavailable, details) cygrpc.StatusCode.unimplemented, details)
return None return None
elif state.client is _CANCELLED: elif state.client is _CANCELLED:
return None return None
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please to comment