backend-hyper: do not disable connection pooling, fix missing ProtocolVersion in ID response

ferristseng / rust-ipfs-api

IPFS HTTP client in Rust

Apache License 2.0

246 stars 68 forks source link

backend-hyper: do not disable connection pooling, fix missing ProtocolVersion in ID response #133

Closed mrd0ll4r closed 12 months ago

mrd0ll4r commented 1 year ago

Fixes #132 Fixes #134

ferristseng commented 1 year ago

Looks there is a clippy issue, but I can merge after that is fixed. Thanks!

iamjpotts commented 1 year ago

The clippy error that's causing the build to fail will be probably be fixed by a rebase. That problem was in master and I had to fix it in my recently merged PR also.

mrd0ll4r commented 1 year ago

Rebased, let's see if that fixes it :)

mrd0ll4r commented 1 year ago

I also added the Protocols field, see here, because why not... I'm not sure if either Protocols or Addresses is nullable, but it probably doesn't hurt to guard against it :)