with_http_retries wasn't raising after the final failure

This oversight is causing `no implicit conversion of nil into String` to overshadow the actual error message in our logs.
This is raising a different exception than you think. `with_http_retries` is private so the error being raised is `NoMethodError`.