Remy Maucherat <> changed:

           What    |Removed                     |Added
                 OS|                            |All

--- Comment #3 from Remy Maucherat <> ---
Do you imply what the testPost test does is incorrect ? The amount of allocated
memory is as intended, and it's not too extreme (IMO). Maybe the memory could
be preallocated using ByteArrayOutputStream(int size) ? I don't think it would
make a big difference, this is only a test, not "best practices".

The test does work for me [so far]. As with the rather equivalent HTTP/2 TLS
test, it is meant to test large POSTs with TLS since that's what people often
complain about.

The test was included in 9.0.33 already (since there was a regression) and
there are no relevant changes in 9.0.34 (only NIO got a minor change, but you
are saying it works). Why did it work then ?

You are receiving this mail because:
You are the assignee for the bug.
To unsubscribe, e-mail:
For additional commands, e-mail:

Reply via email to