Repository: seanmonstar/reqwest (language: rust)

Issue:
Update version of cookie_store
To avoid duplicate dependency on `idna` in tree:

current cargo tree -d output (trust-dns-proto is already at idna v0.3.0 on main):
```
cargo tree -d --all-features
    Updating crates.io index
  Downloaded cookie_store v0.16.1
  Downloaded 1 crate (29.1 KB) in 1.15s
idna v0.2.3
├── cookie_store v0.16.1
│   └── reqwest v0.11.12 
└── trust-dns-proto v0.22.0
    └── trust-dns-resolver v0.22.0
        └── reqwest v0.11.12

idna v0.3.0
├── publicsuffix v2.2.3
│   └── cookie_store v0.16.1 (*)
└── url v2.3.1
    ├── cookie_store v0.16.1 (*)
    ├── reqwest v0.11.12
    └── trust-dns-proto v0.22.0 (*)
```


Discussion / hints:
Looks like there are a number of issues here:
* `time` has an MSRV of `1.60` in the version depended on by `cookie_store` `0.19`
* It looks like `cooke_store` `0.19` doesn't actually compile - I'll look into that first and leave this as draft for now.
* This actually relates to https://github.com/seanmonstar/reqwest/pull/1449 as that will need a similar version of `time` and hence likely the `1.60` MSRV bump

I'll investigate further and leave this in draft for now, but it may be completed anyway by #1449 in which case this can just be closed
Can you elaborate on the `cookie_store` not compiling issue; are you referring to the `reqwest` CI builds, and the fact the latest `cookie_store`, which depends on a more recent `time`, violates the MSRV 1.56.0 build?

Otherwise, it is currently passing the `cookie_store` crate itself is passing its usual CI checks, although admittedly there is not an exhaustive matrix of build configurations, so I am wondering if there is a particular `cookie_store 0.19.0` build you are trying that is failing?
My apoloiges @pfernie - looks like the compile error was due to a corrupted cargo registry on my device. I've now enabled the `wasm-bindgen` feature of `cookie_store` also.

> and the fact the latest cookie_store, which depends on a more recent time, violates the MSRV 1.56.0 build?

I've taken this from the release notes at https://github.com/time-rs/time/blob/main/CHANGELOG.md#0316-2022-10-24
Please note that I've also edited the `ci.yml` to test on `1.60` instead of `1.56` - this should not be merged until it is desired to increase the MSRV to `1.60`
Can you resolve the conflicts please and maybe split the msrv bump into a separate commit?

Fix patch:
diff --git a/Cargo.toml b/Cargo.toml
index 6e07cd696..c9a0521fc 100644
--- a/Cargo.toml
+++ b/Cargo.toml
@@ -127,8 +127,8 @@ rustls-native-certs = { version = "0.6", optional = true }
 rustls-pemfile = { version = "1.0", optional = true }
 
 ## cookies
-cookie_crate = { version = "0.16", package = "cookie", optional = true }
-cookie_store = { version = "0.16", optional = true }
+cookie_crate = { version = "0.17.0", package = "cookie", optional = true }
+cookie_store = { version = "0.20.0", optional = true }
 
 ## compression
 async-compression = { version = "0.4.0", default-features = false, features = ["tokio"], optional = true }


Test patch: