fix: rustls 0.23.1, next version of tokio-rustls

[pimeys]

Hey,

We can merge this only when the next version of tokio-rustls is out. I'm just opening it up already because we upgraded our whole project and need this dependency. Having this PR in our Cargo.toml so we can review and step back to crates.io when the dependency issues in the ecosystem are all merged and published.

Supersedes: #106

[diptanu]

@pimeys Can we merge this now? We are stuck with an older version of rusttls because of this.

[MaxFangX]

@diptanu you can integrate the changes from this PR into your own project with a patch like so:

[dependencies]
axum-server = "=0.6.0"

[patch.crates-io]
axum-server = { git = "https://github.com/grafbase/axum-server", branch = "rustls-0.23" }

More info on patches

[diptanu]

@MaxFangX Thanks! It would be great if this PR was merged in because SSL and being able to use latest version of the rust-tls library would be great.

[julianh-y]

+1 on getting this merged.

[atezet]

In the meantime tokio-rustls = 0.26.0 was released and we got RUSTSEC-2024-0336

@pimeys will you update this PR?

@programatik29 can we get this merged somehow?

[liningpan]

Before this can be merged, I guess we need to figure out what to do with rustls changing the default crypto provider from ring to aws-lc-rs. So far both reqwest and tonic decided to keep using ring as the default for various reasons

• Upgrade to hyper-rustls 0.27/rustls 0.23 seanmonstar/reqwest#2225
• Upgrade to Hyper 1.0 & Axum 0.7 hyperium/tonic#1670 (comment)
• feat: upgrade to rustls 0.23 seanmonstar/reqwest#2299
• Feature request: a way to set/get default ticketer dynamically rustls/rustls#1876 (comment)

One of the reasons listed was aws-lc-rs requires CMake to compile. This was later fixed in aws/aws-lc-rs#317.

[pimeys]

Yep. We also have a similar issue in our workspace with the changed tls impl, it's kind of annoying to set to ring if the workspace is large

[julianh-y]

IMHO, axum-server should stick with the default provider aws-lc-ls now that CMake is not a build dependency anymore.

Perhaps update the README with a note about the rustls providers and perhaps some code snippet showing how to set up a provider for your app. i.e:

            // configure default provider for RUSTLS
            rustls::crypto::aws_lc_rs::default_provider()
                .install_default()
                .expect("install rustls default crypto provider");

[programatik29]

#124 should close this.