kubernetes/ingress-nginx

Basic Auth: deprecation of DES crypt

Aberta

#7.251 aberto em 18 de jun. de 2021

Ver no GitHubGuia do colaborador
 (12 comentários) (0 reação) (0 responsável)Go (7.888 forks)batch import
help wantedkind/featurepriority/important-longterm

Métricas do repositório

Stars
 (15.199 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

When using a RHEL / CentOS / UBI 7 or 8 base image build of nginx several of the e2e tests fail due to DES deprecation with crypt.

ingress-nginx should default to SHA hashed passwords for basic auth, currently hashes generated from openssl passwd -crypt can replace -crypt with -6, and the following line should indicate a different hashing method in the salt, like foo:$6$

https://github.com/kubernetes/ingress-nginx/blob/b1c8e3047ba31d7ea78f6e0915d187db75230ba5/test/e2e/annotations/auth.go#L210

Without specifying a different hashing method in the salt the test will fail with a 500 code and a log message error like crypt_r() failed (22: Invalid argument) when using of the the above mentioned base images.

If the approach with updating the tests sounds acceptable I would be happy to submit a PR.

/kind feature

Guia do colaborador

Direção de pesquisa
Inspecione o arquivo de teste e2e em test/e2e/annotations/auth.go linha 210 e substitua openssl passwd crypt por openssl passwd 6 para gerar hashes SHA 512. Atualize o salt para começar com $6$.
Pilha de tecnologia
godocker
Domain
backendtesting
Tipo Issue
Teste
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GoKubernetese2e testing
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70