jnh-connector: Fixed issue where FirstByteCachingStream failed to inc… #5877
Conversation
…lude the buffered byte in returned data length when reading with array
|
for me it looks good, thanks, however you should update the copyright year in both files (change the last year to the |
|
Copyright years updated, Eclipse account created and ECA signed :-) |
|
it looks better and better, thank you! The copyright years are fixed, thanks a lot. The ECA is still not recognized. Could you please check the Git ECA validation status page for this PR and follow the troubleshooting guide to see what could possibly be wrong with it? On the same page, you can re-validate the ECA for the PR. If it's required (for the ECA validation), you can amend existing commits and force push them back into the PR branch to fix the ECA issue. But I hope it will be easier to solve. Thanks a lot! |
lbihanic
force-pushed
the
jnh-length-error-fix
branch
from
5380e95 to
2fca287
Compare
3 participants
Fixed issue where FirstByteCachingStream fails to include the buffered byte in returned data length when reading with array.
When the problem occurred, the returned length was 1 byte less than the actual array content and the consumer failed to "see" the whole response data. Detected through Jackson parse error reporting missing closing '}'.