-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Proxy authentication does not work with vertx client and https endpoint #6350
Comments
Didn't we have a generic proxy (spec) test for all the clients? |
I need to be more specific, the problem appears to be with https handling. Specifically under the covers okhttp is creating the https tunnel starting with a connect request: https://github.com/square/okhttp/blob/06644bb0507873e9a3b89d9107da537f1b140e91/okhttp/src/main/java/okhttp3/internal/connection/RealConnection.java#L444 For the other client types if we are associating the authentication info just with the header, the similar logic they have for this case doesn't pick up the username / password. |
A guide to our proxy scenarios:
socks 4/5 are also untested. |
closes: fabric8io#6350 Signed-off-by: Steve Hawkins <shawkins@redhat.com>
closes: fabric8io#6350 Signed-off-by: Steve Hawkins <shawkins@redhat.com>
closes: fabric8io#6350 Signed-off-by: Steve Hawkins <shawkins@redhat.com>
closes: fabric8io#6350 Signed-off-by: Steve Hawkins <shawkins@redhat.com>
closes: fabric8io#6350 Signed-off-by: Steve Hawkins <shawkins@redhat.com>
closes: fabric8io#6350 Signed-off-by: Steve Hawkins <shawkins@redhat.com>
Describe the bug
The code is using the generic proxy authentication, but vertx needs the username and password passed to it via the proxy options.
Fabric8 Kubernetes Client version
6.13.3
Steps to reproduce
Use proxy.username and proxy.password with the vertx client
Expected behavior
Proxy authentication should work
Runtime
Kubernetes (vanilla)
Kubernetes API Server version
1.25.3@latest
Environment
Linux
Fabric8 Kubernetes Client Logs
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: