Remove reqwest dependency

[KendallWeihe]

No description provided.

[KendallWeihe]

Converting to a draft for the time being for further consideration

[KendallWeihe]

I think we'll probably move forward with this, but @diehuxx wondering if you can give it a look. Windows test seems to be failing so I'm guessing something in our TLS or TCP (or something else?) is causing issues specifically on Windows, so I'll get to the bottom of that.

I may rework the http.rs module to be a slightly different design, but I wanted to spike in the ground a functional solution where all tests are passing (except apparently Windows?).

The major win here is to get us off of OpenSSL entirely. Also it reduces the byte code file size by ~10% which is not insignificant. We're unable to get out of ring showing up in our Cargo.lock file, but @finn-tbd just recently added CI jobs which will fail if it slips into the compiled result.

[diehuxx]

@KendallWeihe The failure is that we expect Web5Error::Network("failed to lookup address information") but instead we're getting Web5Error::Network("No such host is known"). All that differs is the text of the error message returned to us by TcpStream::Connect in http.rs.

I suspect that Windows rust implementation of TcpStream::connect returns one error message and Mac and Linux return another. In both cases, the error message conveys the same thing, i.e. This host isn't valid.

I'd be comfortable just updating the test to accept either error message. I added a commit doing this.

[diehuxx]

feel like we could just call this get and it would be less confusing. get_bytes_as_http_response sounds like i'm parsing the url as an http response