kubernetes/ingress-nginx

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

Open

#8.384 aberto em 23 de mar. de 2022

Ver no GitHub
 (17 comments) (2 reactions) (2 assignees)Go (15.199 stars) (7.888 forks)batch import
good first issuehelp wantedkind/featureneeds-triagepriority/backlog

Description

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:

Guia do colaborador

Pilha de tecnologia
gokubernetes
Domain
backendinfrastructure
Tipo Issue
feature
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
half day
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
blocked
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
Go programmingKubernetes basicsingress nginx controller architecture
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
25
Direção de pesquisa
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.