Thanks for the PR! Looks like cookie_store is using the brand new TryFrom traits, making this fail reqwest's minimum rust version check. Hm, while I've generally cared much less about increasing that, TryFrom has been stable for less than a month! I'm not sure which way to fall, yet....

Yeah saw that, unfortunate and tricky with bleeding edge Rust usage in a small crate that big stable dependencies like reqwest depends on. And tricky again that this upgrade was specifically required due to cookie using ring which there can only be one of. Not fully ideal.

I do think your suggestion of splitting cookie into a separate cookie-secure that has the ring dependency would be best (rwf2/cookie-rs#120). Or if cookie could use a Rust-only crate instead of ring (rwf2/cookie-rs#113). or both.

I have no issue dropping cookie_store back to using the try_from crate to avoid the minimum version bump. Let me know your inclination and I can cut a new release if that’s preferred.

@pfernie if you wouldn't mind, that'd help a lot! As I said, I'm fine with minimum rust version bumps usually, it's just quite likely that many people haven't yet upgraded to 1.34, as it's under a month old. After a few months, I would think nothing of it.

cookie_store v0.7.0 has been published, which reverts to using try_from crate for now.

Passes the checks now, merge in?

Thanks! (Sorry, I try to disconnect on weekends.)