kubernetes/ingress-nginx

Feature Request: Allow disabling custom-http-errors per ingress

Open

Aperta il 23 mar 2022

Vedi su GitHub
 (17 commenti) (2 reazioni) (2 assegnatari)Go (15.199 star) (7888 fork)batch import
good first issuehelp wantedkind/featureneeds-triagepriority/backlog

Descrizione

When nginx.ingress.kubernetes.io/custom-http-errors is set to string off, overwrite the configmap value with an empty list instead of skipping it.

My current work-around is to use the 418 HTTP code (nginx.ingress.kubernetes.io/custom-http-errors: "418") because my apps aren't using it :wink:

Guida contributor

Tech stack
gokubernetes
Dominio
backendinfrastructure
Tipo issue
feature
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
half day
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
blocked
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
Go programmingKubernetes basicsingress nginx controller architecture
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
25
Direzione di ricerca
Investigate the annotation parsing code for custom http errors, likely in internal/ingress/annotations/custom http errors. Modify the logic to treat the string 'off' as an empty list, overriding the configmap default. Check comments and linked PRs for prior discussion and ensure backward compatibility. Write tests for the new behavior.