adutra commented on code in PR #631:
URL: https://github.com/apache/polaris/pull/631#discussion_r1909123471
##########
dropwizard/service/src/test/java/org/apache/polaris/service/dropwizard/PolarisApplicationIntegrationTest.java:
##########
@@ -722,7 +722,7 @@ public void testRequestBodyTooLarge() {
.header(REALM_PROPERTY_KEY, realm)
.post(largeRequest)) {
assertThat(response)
- .returns(Response.Status.BAD_REQUEST.getStatusCode(),
Response::getStatus)
+ .returns(Response.Status.REQUEST_ENTITY_TOO_LARGE.getStatusCode(),
Response::getStatus)
Review Comment:
Sure, but from [RFC
9112](https://httpwg.org/specs/rfc9112.html#persistent.tear-down):
> If a server performs an immediate close of a TCP connection, there is a
significant risk that the client will not be able to read the last HTTP
response. If the server receives additional data from the client on a fully
closed connection, such as another request sent by the client before receiving
the server's response, the server's TCP stack will send a reset packet to the
client; unfortunately, the reset packet might erase the client's unacknowledged
input buffers before they can be read and interpreted by the client's HTTP
parser.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
