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

no_proxy ipv6 entries not handled correctly #6781

Closed
shawkins opened this issue Jan 10, 2025 · 7 comments · Fixed by #6782
Closed

no_proxy ipv6 entries not handled correctly #6781

shawkins opened this issue Jan 10, 2025 · 7 comments · Fixed by #6782
Assignees
Milestone

Comments

@shawkins
Copy link
Contributor

Describe the bug

ipv6 entries that appear in the no_proxy list are not checked.

Fabric8 Kubernetes Client version

6.11.0

Steps to reproduce

set NO_PROXY to contain an ipv6 entry 2048::/110

Expected behavior

Addresses within the range should not use a proxy

Runtime

Kubernetes (vanilla)

Kubernetes API Server version

1.25.3@latest

Environment

Linux

Fabric8 Kubernetes Client Logs

No response

Additional context

No response

shawkins added a commit to shawkins/kubernetes-client that referenced this issue Jan 10, 2025
shawkins added a commit to shawkins/kubernetes-client that referenced this issue Jan 10, 2025
shawkins added a commit to shawkins/kubernetes-client that referenced this issue Jan 10, 2025
@manusa manusa added this to the 7.1.0 milestone Jan 13, 2025 — with automated-tasks
manusa pushed a commit that referenced this issue Jan 13, 2025
@shawkins
Copy link
Contributor Author

@manusa I think we will need this for our next release of Keycloak, so we'll probably need this in Quarkus 3.15.x.

cc @metacosm @vmuzikar

@manusa
Copy link
Member

manusa commented Jan 23, 2025

Should we backport the fix to 6.13 then?

(I'm assuming this based on https://github.com/quarkusio/quarkus/blob/7c367ac49c2f617b6e348247d5b219e91a993f34/pom.xml#L72)

@shawkins
Copy link
Contributor Author

Should we backport the fix to 6.13 then?

Yes, I believe that is still the target release.

@vmuzikar
Copy link

We need it in Quarkus 3.20, as IPv6 isn't coming until Keycloak 26.2.

@manusa
Copy link
Member

manusa commented Jan 24, 2025

So no back-port + release needed then?

If that's the case, I can prioritize releasing 7.1 (which was the original plan for next week).

Also, @shawkins, I'd need your input for #6792, to see if we need to do something there or leave it as is.

@vmuzikar
Copy link

So no back-port + release needed then?

I'm not sure which version Quarkus 3.20, especially JOSDK will use. CC @metacosm

@metacosm
Copy link
Collaborator

So no back-port + release needed then?

I'm not sure which version Quarkus 3.20, especially JOSDK will use. CC @metacosm

QOSDK 7 will be used in Quarkus 3.20 (Fabric8 client 7 is used in Quarkus starting with 3.18). QOSDK 7 will use JOSDK 5 which has breaking API changes (though they shouldn't be too difficult to adapt for).

manusa pushed a commit to manusa/kubernetes-client that referenced this issue Jan 28, 2025
closes: fabric8io#6781

Signed-off-by: Steve Hawkins <[email protected]>
Signed-off-by: Marc Nuri <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants