Skip to content
This repository was archived by the owner on Jul 31, 2023. It is now read-only.

Commit 19a9151

Browse files
lwcsongy23
authored andcommitted
Map 422 Unprocessable Entity to INVALID_ARGUMENT (#1149)
1 parent df42942 commit 19a9151

2 files changed

Lines changed: 5 additions & 0 deletions

File tree

plugin/ochttp/trace.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -186,6 +186,8 @@ func TraceStatus(httpStatusCode int, statusLine string) trace.Status {
186186
code = trace.StatusCodeCancelled
187187
case http.StatusBadRequest:
188188
code = trace.StatusCodeInvalidArgument
189+
case http.StatusUnprocessableEntity:
190+
code = trace.StatusCodeInvalidArgument
189191
case http.StatusGatewayTimeout:
190192
code = trace.StatusCodeDeadlineExceeded
191193
case http.StatusNotFound:

plugin/ochttp/trace_test.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -668,6 +668,9 @@ func TestStatusUnitTest(t *testing.T) {
668668
{204, trace.Status{Code: trace.StatusCodeOK, Message: `OK`}},
669669
{100, trace.Status{Code: trace.StatusCodeUnknown, Message: `UNKNOWN`}},
670670
{500, trace.Status{Code: trace.StatusCodeUnknown, Message: `UNKNOWN`}},
671+
{400, trace.Status{Code: trace.StatusCodeInvalidArgument, Message: `INVALID_ARGUMENT`}},
672+
{422, trace.Status{Code: trace.StatusCodeInvalidArgument, Message: `INVALID_ARGUMENT`}},
673+
{499, trace.Status{Code: trace.StatusCodeCancelled, Message: `CANCELLED`}},
671674
{404, trace.Status{Code: trace.StatusCodeNotFound, Message: `NOT_FOUND`}},
672675
{600, trace.Status{Code: trace.StatusCodeUnknown, Message: `UNKNOWN`}},
673676
{401, trace.Status{Code: trace.StatusCodeUnauthenticated, Message: `UNAUTHENTICATED`}},

0 commit comments

Comments
 (0)