dariusc93 / rust-igd

Internet Gateway Device (UPNP) client

Home Page:https://docs.rs/igd/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

`attohttpc` is a thread-blocking HTTP client

thomaseizinger opened this issue · comments

As far as I understand, https://docs.rs/attohttpc is a blocking HTTP client and doesn't use async. Now that we have integration with two runtimes, we should probably use an async client or hyper directly.

See https://github.com/dariusc93/rust-igd/blob/master/src/gateway.rs#L28-L32.

can we feature gate the whole blocking mod? If you want to make it non (or less as someone might be misusing the default-features) breaking you can add it to the default-features

Ah, I had no idea this code was duplicated with the non-blocking mode! Yeah feature-gating would be ideal to reduce our dependency tree :)

Ah, I had no idea this code was duplicated with the non-blocking mode! Yeah feature-gating would be ideal to reduce our dependency tree :)

yeah we are using aio::gateway :)

Nice! I guess this issue can be seen as implementing the feature flag then :)