Defer closing body before checking HTTP status code

[nikpivkin]

This is a quote from the documentation of the net/http package:

When err is nil, resp always contains a non-nil resp.Body. Caller should close resp.Body when done reading from it.

In the current implementation, if the error code is 404, the response body will not close.

[codecov]

Codecov Report

Merging #2870 (0d2e4db) into master (899235e) will increase coverage by 0.00%.
Report is 2 commits behind head on master.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2870   +/-   ##
=======================================
  Coverage   98.06%   98.07%           
=======================================
  Files         138      139    +1     
  Lines       12327    12342   +15     
=======================================
+ Hits        12089    12104   +15     
  Misses        162      162           
  Partials       76       76           

see 2 files with indirect coverage changes

[gmlewis]

Thank you, @nikpivkin !
LGTM.

Awaiting second LGTM+Approval from any other contributor to this repo before merging.

[vandanrohatgi]

Makes sense. LGTM!

[gmlewis]

Thank you, @vandanrohatgi ! ❤️
Merging.