Include the full discovery url on invalid HTTP response

[Timshel]

Hey,

Simple PR to add the full discovery url to the error since in lot of cases it will be due to an improper issuer_url.

[codecov]

Codecov Report

Attention: 9 lines in your changes are missing coverage. Please review.

Comparison is base (d796cca) 51.84% compared to head (c706e10) 51.76%.

Files	Patch %	Lines
src/discovery.rs	0.00%	9 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #142      +/-   ##
==========================================
- Coverage   51.84%   51.76%   -0.08%     
==========================================
  Files          16       16              
  Lines        2145     2148       +3     
==========================================
  Hits         1112     1112              
- Misses       1033     1036       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ramosbugs]

Thanks for the PR! I'll merge once CI passes (looks like cargo fmt is unhappy)

[Timshel]

O yes sorry :)

[Timshel]

I had tried running it and it failed so I invalidly concluded that you did not use it.
On another project it's included in the rust-toolchain.toml.

But I'm a bit of a rust noob so not sure of the best practices :).

[ramosbugs]

Thanks!

I had tried running it and it failed so I invalidly concluded that you did not use it.

When I run it locally with Cargo/Rust 1.71.0 (rustfmt 1.5.2-stable), it doesn't seem to change any of the formatting. Is it possible you ran it with the nightly toolchain?

On another project it's included in the rust-toolchain.toml.

I think this file is mainly for pinning to specific versions: https://rust-lang.github.io/rustup/overrides.html#the-toolchain-file.

[ramosbugs]

This is now released in 3.5.0.