RFR: 8263031: HttpClient throws Exception if it receives a Push Promise that is too large [v6]

Conor Cleary ccleary at openjdk.java.net
Mon Mar 28 10:26:25 UTC 2022


On Fri, 25 Mar 2022 15:55:54 GMT, Conor Cleary <ccleary at openjdk.org> wrote:

>> Oh yes, good point. I think `ErrorFrame.PROTOCOL_ERROR` would be the most appropriate here. I'll amend the change accordingly.
>
> However, I'll look into the specification further for the other cases and see if they need be changed as well. Though closing the whole connection with `GoAwayFrame` seems correct

I changed three more occurences to `ErrorFrame.PROTOCOL_ERROR`, its easier to understand and lines up more clearly with specification

-------------

PR: https://git.openjdk.java.net/jdk/pull/7696


More information about the net-dev mailing list