Skip to content

replace hyper-socks2 with hyper-util client-proxy feature#1795

Merged
clux merged 1 commit into
kube-rs:mainfrom
tottoto:use-hyper-util-client-proxy-feature
Jul 27, 2025
Merged

replace hyper-socks2 with hyper-util client-proxy feature#1795
clux merged 1 commit into
kube-rs:mainfrom
tottoto:use-hyper-util-client-proxy-feature

Conversation

@tottoto
Copy link
Copy Markdown
Contributor

@tottoto tottoto commented Jul 22, 2025

Motivation

hyper-util provides client-proxy feature.

Solution

Uses hyper-util client-proxy feature.

@tottoto tottoto force-pushed the use-hyper-util-client-proxy-feature branch from 57262fb to bdfa532 Compare July 22, 2025 21:08
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 22, 2025

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Project coverage is 75.1%. Comparing base (6590ea9) to head (92a2ac0).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
kube-client/src/client/builder.rs 0.0% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##            main   #1795     +/-   ##
=======================================
- Coverage   75.2%   75.1%   -0.0%     
=======================================
  Files         84      84             
  Lines       7791    7791             
=======================================
- Hits        5853    5850      -3     
- Misses      1938    1941      +3     
Files with missing lines Coverage Δ
kube-client/src/client/builder.rs 64.3% <0.0%> (ø)

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tottoto tottoto force-pushed the use-hyper-util-client-proxy-feature branch from bdfa532 to 92a2ac0 Compare July 23, 2025 10:08
@tottoto tottoto force-pushed the use-hyper-util-client-proxy-feature branch from 92a2ac0 to 0f28ad7 Compare July 26, 2025 23:39
Copy link
Copy Markdown
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.

Thank you. More trustworthy dependencies.

Did a quick local test with a local socks5 proxy over ssh and it worked as before.

@clux clux added the changelog-change changelog change category for prs label Jul 27, 2025
@clux clux added this to the next-major milestone Jul 27, 2025
@clux clux merged commit 34d6d05 into kube-rs:main Jul 27, 2025
15 of 16 checks passed
@tottoto tottoto deleted the use-hyper-util-client-proxy-feature branch July 27, 2025 10:45
@clux clux changed the title use hyper-util client-proxy feature replace hyper-socks2 with hyper-util client-proxy feature Sep 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-change changelog change category for prs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants