dmfs / http-client-essentials-suite

A lightweight http-client suite.
Apache License 2.0
11 stars 5 forks source link

tolerate absent getContentLengthLong method, fixes #114 #115

Closed dmfs closed 6 years ago

dmfs commented 6 years ago

There doesn't seem to be an easy check for the presence of a method. So we just fall back to getContentLenght if getContentLengthLong doesn't exist.

codecov[bot] commented 6 years ago

Codecov Report

Merging #115 into master will decrease coverage by 0.06%. The diff coverage is 50%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #115      +/-   ##
============================================
- Coverage     67.43%   67.37%   -0.07%     
  Complexity      488      488              
============================================
  Files           148      148              
  Lines          1686     1689       +3     
  Branches        112      112              
============================================
+ Hits           1137     1138       +1     
- Misses          496      498       +2     
  Partials         53       53
Impacted Files Coverage Δ Complexity Δ
...urlconnection/HttpUrlConnectionResponseEntity.java 53.84% <50%> (-6.16%) 4 <1> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 0ddee05...ab8a7fd. Read the comment docs.