Invalid URL Generation Due to Invalid Character in Path

[leonardo-zup]

Describe the bug

The line 3304 of the file kubernetes/src/main/java/io/kubernetes/client/openapi/apis/CustomObjectsApi.java:

3304 │ ········String localVarPath = "/apis/{group}/{version}/{plural}#\u{200e}"

is causing the generation of an invalid URL. The inclusion of #\u{200e} at the end of the URL path results in an invalid or improperly formatted URL.

Client Version
22.0.0

Kubernetes Version
Not applicable – the Kubernetes version does not affect this issue.

Java Version
Not applicable – the Java version does not affect this issue.

To Reproduce

• Call the listCustomObjectForAllNamespacesCall method with a labelSelector parameter.
• Observe that the labelSelector parameter is not included in the generated URL due to the invalid character #\u{200e}.

Expected behavior
The URL should correctly include all relevant parameters, such as labelSelector.

KubeConfig
Not applicable.

Server (please complete the following information):
Not applicable – the Server does not affect this issue.

[brendandburns]

I don't see this in the code at release-22

https://github.com/kubernetes-client/java/blob/release-22/kubernetes/src/main/java/io/kubernetes/client/openapi/apis/CustomObjectsApi.java

Do you have a link to the commit where you see this code?

Either way, this comes out of the open API spec so if there is a bug, it needs to be corrected in that spec.

Given that HEAD looks ok:
https://github.com/kubernetes-client/java/blob/master/kubernetes/src/main/java/io/kubernetes/client/openapi/apis/CustomObjectsApi.java#L3304

I don't think that there is any action needed here, but let me know if there is more to do.

[leonardo-zup]

On the screenshot you can see the #

The hidden char you can only see downloading the code and using bat -A CustomObjectsApi.java or something similar

[leonardo-zup]

[yue9944882]

https://github.com/kubernetes-client/gen/blame/master/openapi/custom_objects_spec.json#L47

This was introduce from the swagger model here ^ so it will impact all language clients we have. Will send a PR to fix the model first..

[brendandburns]

Thanks @yue9944882. Yeah we'll need to fix that spec file first and then regenerate the code.

[k8s-triage-robot]

The Kubernetes project currently lacks enough contributors to adequately respond to all issues.

This bot triages un-triaged issues according to the following rules:

• After 90d of inactivity, lifecycle/stale is applied
• After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
• After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

• Mark this issue as fresh with /remove-lifecycle stale
• Close this issue with /close
• Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

[leonardo-zup]

The issue was fixed on the latest version.

/close