Can we use 308 for permanent-redirects?

[Stono]

Hey!

As you can see from this line:

ingress-nginx/internal/ingress/annotations/redirect/redirect.go

Line 83 in cdbb9bd

Code: http.StatusMovedPermanently,

Requests configured with nginx.ingress.kubernetes.io/permanent-redirect, are being redirected with a 301.

I feel a 308 would be more appropriate https://httpstatuses.com/308 as this minimises the risk of clients changing POST to GET.

Ta!

[aledbf]

@Stono yes but we need to make this configurable like in the configmap https://kubernetes.github.io/ingress-nginx/user-guide/nginx-configuration/configmap/#http-redirect-code

[fejta-bot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale