SmartHttpFetchConnection: suppress errors on close()HEADmaster

Suppress errors on closing the input stream in a fetch. If the fetch
itself worked and didn't throw an exception, then we don't care about
exceptions on closing the input stream. Just log the exception at debug
level.

It appears that in particular some Azure git servers may do things that
cause an exception in Apache HTTP on closing its ChunkedInputStream.[1]
The JDK's equivalent ChunkedInputStream ignores exceptions at that
place,[2] which explains why fetching with the JDK HTTP support works,
while it sometimes fails with Apache HTTP.

[1] https://github.com/apache/httpcomponents-core/blob/a5c117028b7c620974304636d52f06f172f1d08b/httpcore/src/main/java/org/apache/http/impl/io/ChunkedInputStream.java#L306
[2] https://github.com/openjdk/jdk/blob/1a206d2a6cade07249f6922072ac9d29aa56bc43/src/java.base/share/classes/sun/net/www/http/ChunkedInputStream.java#L785

Bug: jgit-59
Change-Id: Ic26f0188f4f2ed9318f01ccfe2f93c3596950197

0 files changed, 0 insertions, 0 deletions