Client does not report expected error code for 431 HTTP status

[jhump]

A well-formed gRPC response always has an HTTP status code of “200 OK”. The gRPC docs specify a table for mapping other HTTP status codes to gRPC error codes: https://github.com/grpc/grpc/blob/master/doc/http-grpc-status-mapping.md.

The client in this repo mostly conforms to this table. This was tested with responses that include a variety of HTTP status codes but no body (and thus no “content-type” header) and no “grpc-status” header. The status codes tested include all values in the table linked above as well as a sampling of other codes (to test the last line of the table, which states that all other HTTP status codes should map to an unknown error code).

The one issue uncovered was for HTTP status 431 (Request Header Fields Too Large). Per the table, this HTTP status should result in an unknown error code. However it instead results in an internal error code.

[ejona86]

Heh. There's a TODO saying to update the spec.

grpc-java/core/src/main/java/io/grpc/internal/GrpcUtil.java

Line 309 in 781b4c4

// TODO(carl-mastrangelo): this should be added to the http-grpc-status-mapping.md doc.

It was added in d5eb248

I suspect that was because the server-side behavior changed from a RST_STREAM to HEADERS with error. It would be fair to discuss cross-language what code is appropriate and use it instead of unknown.