Reqwest: Re-enable trust-dns feature

[seanmonstar]

The trust-dns feature was disabled in #617 as the dependency was not yet updated to std::future. We should re-enable once it is available.

[daxpedda]

This was fixed here: https://github.com/bluejekyll/trust-dns/pull/849.

EDIT: So basically from version 0.18.0-alpha.1.

[seanmonstar]

We can re-enable once the version is out of alpha!

[seanmonstar]

Some notes as I tried to re-enable this:

• ResolveError implements Fail instead of std::error::Error. I can get around this since it also implements Into<io::Error>, but it'd be better to not include failure in the dependency tree.
• AsyncResolver requires passing a tokio::runtime::Handle to its constructors, but we don't have one and have no way of materializing one.

[daxpedda]

• ResolveError implements Fail instead of std::error::Error. I can get around this since it also implements Into<io::Error>, but it'd be better to not include failure in the dependency tree.

It still implements Display, wihch is what we are using as far as I can see:
https://github.com/seanmonstar/reqwest/blob/35c6ddd00b20243106e11ca9aa46abbc33d56bc9/src/dns.rs#L73
In any case, trust-dns pulls in failure anyway, so no loss there.

• AsyncResolver requires passing a tokio::runtime::Handle to its constructors, but we don't have one and have no way of materializing one.

See https://github.com/bluejekyll/trust-dns/issues/977 and https://github.com/tokio-rs/tokio/pull/2040.

I would like to take a stab at this when the Handle is resolved, if thats alright.

[bluejekyll]

@seanmonstar, we can remove all depdencies on Fail and failure. We'll lose backtrace, but that is coming in std.

might need a 0.19 release of trust-dns...

[bluejekyll]

FYI: https://github.com/bluejekyll/trust-dns/pull/988

This drops failure, uses thiserror for some std error generation. Is there anything else? I’m planning to do a 0.19 release this weekend to get this out.

[bluejekyll]

Ok, 0.19 is released.