David Ramos

That's a requirement I haven't seen before! Your approach of providing a custom HTTP client to wrap reqwest is exactly what I'd suggest. Is the clone needed though? It seems...

I've just merged 23b952b2, which in 5.0 will allow stateful HTTP clients like a `reqwest::Client` to passed in instead of a function. This should make it easy to set the...

sounds good! let's do this once `reqwest` supports 1.0, and that will require a major version bump of this crate and the downstream `openidconnect` crate.

I've started merging breaking changes for the next major release (5.0) of this crate into `main`. Hopefully this change will be done soon enough to make it into 5.0 (no...

> It's been merged! [seanmonstar/reqwest#2039](https://github.com/seanmonstar/reqwest/issues/2039) I saw! I should have a branch ready soon, but I won't be able to cut a release until `reqwest` 0.12 is released with the...

Alright, I've upgraded to `http` 1.0 in the `http-1.0` branch. Until a release is cut, anyone can test it as follows: ```toml oauth2 = { git = "https://github.com/ramosbugs/oauth2-rs.git", rev =...

> @ramosbugs what would be the minimum required Rust compiler version for this? should still be 1.65 > Testing done, and this is working. Please merge still waiting on a...

[5.0.0-alpha.3](https://crates.io/crates/oauth2/5.0.0-alpha.3) is now released with dependencies on `http` 1.0 and `reqwest` 0.12

fixed, thanks! 🤦‍♂️

Hey @MarijnS95, My concern with using `http`'s request/response types is that they include a lot more functionality than the barebones types defined by this crate. Different HTTP client implementations may...