Close okhttp response

[honnix]

Hey, I just made a Pull Request!

Description

Close okhttp response for requests we don't expect response body payload.

Motivation and Context

An OKHttp response must be closed (as explained here). For those requests we expect body payload, we read the body stream which results in close automatically, but for those we don't expect response body payload, currently we are not closing the response, which for some condition results in leaving threads hanging around.

Have you tested this? If so, how?

Existing tests.

Checklist for PR author(s)

• Changes are covered by unit test
• All tests pass
• Code coverage check passes
• Error handling is tested
• Errors are handled at the appropriate layer
• Errors that cannot be handled where they occur are propagated
• (optional) Changes are covered by system test
• Relevant documentation updated
• This PR has NO breaking change to public API
• This PR has breaking change to public API and it is documented

Checklist for PR reviewer(s)

• This PR has been incorporated in release note for the coming version
• Risky changes introduced by this PR have been all considered