Azure / Azure/aks-app-routing-operator
annotation permanent-redirect do not support '%20'
- Dominant language
- Go
- Stars
- 40
- Forks
- 31
- PR merge metrics
- No merged PRs in 30d
Description
Hello Y'all,
We were migrating from on-prem K8s to AKS. As we started to use "Managed NGINX ingress with the application routing add-on" we ran in to some issues with annotation validation.
For example:
if I set permanent-redirect value to something that contains '%20' I receive and error - validators.go:237] validation error on ingress prod-ns/redirect-org: annotation permanent-redirect contains invalid value https://super-site.sharepoint.com/sites/teams/IT/devops123/IT%20OPS
This is real example:
```
nginx.ingress.kubernetes.io/permanent-redirect: https://super-site.sharepoint.com/sites/teams/IT/devops123/IT%20OPS
```
Questions:
1. Is there a way to make it work?
2. If not, can annotation validation be disabled for Managed Nginx ingress?
P.S. I have used this article to setup Nginx ingress class and ingress controller - https://learn.microsoft.com/en-us/azure/aks/app-routing-nginx-configuration
P.S.S. Old version of nginx installed from Helm works fine.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start at validators.go:237 and reproduce validation using the permanent-redirect annotation value shown in the issue. Trace how encoded spaces are checked in the Managed NGINX ingress path and compare the behavior with the Helm-installed controller. Done means the supported behavior or a documented configuration path for this annotation is established.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go, kubernetes, nginx
- Domain
- cloud, infrastructure, networking
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100