bump trust-dns-proto trust-dns-resolver

[attila-lin]

No description provided.

[abr-egn]

Thanks for contributing! I've kicked off a test run, assuming that's green I'll merge this in.

[abr-egn]

Unfortunately, it looks like trust-dns-resolver 0.21.1 doesn't compile under our MSRV (1.49):

[2022/03/14 19:26:44.507]    Compiling trust-dns-resolver v0.21.1
[2022/03/14 19:26:44.507] error[E0658]: use of unstable library feature 'clamp'
[2022/03/14 19:26:44.507]    --> /home/ubuntu/.cargo/registry/src/github.com-1ecc6299db9ec823/trust-dns-resolver-0.21.1/src/dns_lru.rs:242:18
[2022/03/14 19:26:44.507]     |
[2022/03/14 19:26:44.507] 242 |                 .clamp(self.negative_min_ttl, self.negative_max_ttl);
[2022/03/14 19:26:44.507]     |                  ^^^^^
[2022/03/14 19:26:44.507]     |
[2022/03/14 19:26:44.507]     = note: see issue #44095 <https://github.com/rust-lang/rust/issues/44095> for more information
[2022/03/14 19:26:44.776] error: aborting due to previous error

[attila-lin]

Hello! Thank you with your reply.

Any plan with the updating of MSRV? I see a lot deps of this crate is outdated.

[abr-egn]

Talked about it with the team and we're going to bump our MSRV to 1.50.0, which should be sufficient for this change. Once that's in we can merge this :)

In the meanwhile, it looks like there are some compile failures unrelated to that:

[2022/03/14 19:19:46.369] error[E0308]: mismatched types
[2022/03/14 19:19:46.369]   --> src/srv.rs:84:17
[2022/03/14 19:19:46.369]    |
[2022/03/14 19:19:46.369] 83 |             let srv = match record.data() {
[2022/03/14 19:19:46.369]    |                             ------------- this expression has type `std::option::Option<&RData>`
[2022/03/14 19:19:46.369] 84 |                 RData::SRV(s) => s,
[2022/03/14 19:19:46.369]    |                 ^^^^^^^^^^^^^ expected enum `std::option::Option`, found enum `RData`
[2022/03/14 19:19:46.369]    |
[2022/03/14 19:19:46.369]    = note: expected enum `std::option::Option<&RData>`
[2022/03/14 19:19:46.369]               found enum `RData`

[attila-lin]

Happy to hear that!

[abr-egn]

It turns out that due to incompatibilities with other dependencies we needed to bump trust-dns-proto and trust-dns-resolver as part of the larger MSRV update (#596), so I'm going to close out this PR. Thanks again for bringing this to our attention!