Azure / Azure/aks-app-routing-operator

annotation permanent-redirect do not support '%20'

Open
#245 3 comments 0 reactions 0 assignees View on GitHub
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.