Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update kube-client's rustls dependency to 0.21.4 #1341

Merged
merged 1 commit into from
Nov 11, 2023
Merged

Update kube-client's rustls dependency to 0.21.4 #1341

merged 1 commit into from
Nov 11, 2023

Conversation

MathiasPius
Copy link
Contributor

@MathiasPius MathiasPius commented Nov 11, 2023

kube-client is incompatible with rustls versions prior to 0.21.4 due to the renaming of rustls::client::builder::ConfigBuilder::.with_client_auth_cert

Fixes #1340

Motivation

It is not currently possible to build kube-client using the stated minimum dependencies in its cargo manifest.

Solution

Updating kube-client's rustls dependency to 0.21.4 which is its actual minimum supported version.

kube-client is incompatible with rustls versions prior to 0.21.4 due to the renaming of rustls::client::builder::ConfigBuilder::.with_client_auth_cert

Fixes #1340

Signed-off-by: Mathias Pius <contact@pius.io>
Copy link

codecov bot commented Nov 11, 2023

Codecov Report

Merging #1341 (49fefa7) into main (5813ad0) will increase coverage by 0.1%.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##            main   #1341     +/-   ##
=======================================
+ Coverage   72.1%   72.2%   +0.1%     
=======================================
  Files         75      75             
  Lines       6377    6377             
=======================================
+ Hits        4597    4598      +1     
+ Misses      1780    1779      -1     

see 1 file with indirect coverage changes

@clux clux added this to the 0.88.0 milestone Nov 11, 2023
@clux clux added the changelog-exclude changelog excluded prs label Nov 11, 2023
Copy link
Member

@clux clux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good fix. Thank you.

Maybe it's worth us considering having a -Zminimal-versions build.

@clux clux merged commit 0abd2bd into kube-rs:main Nov 11, 2023
17 checks passed
@MathiasPius MathiasPius deleted the update-kube-client-rustls-dep branch November 23, 2023 20:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-exclude changelog excluded prs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Minimum supported rustls version is 0.21.4 not 0.21.0
2 participants