Fix unused branches in http_client

cppcheck correctly finds that these branches will always be hit, so the
branch is unneeded.  Let's start by getting the code cleaned up.

Tested: CI only, cpp check.

Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I7b89337a81e676915243d5bc9d26c24a89c74aef