nodejs/node

Run tests with OPENSSL_CONF set to embedded openssl.cnf

Fechada

#24.373 aberto em 15 de nov. de 2018

Ver no GitHubGuia do colaborador
 (3 comentários) (0 reação) (0 responsável)JavaScript (35.535 forks)batch import
help wantedopensslstaletest

Métricas do repositório

Stars
 (117.218 estrelas)
Métricas de merge de PR
 (Mesclagem média 18d 17h) (219 fundiu PRs em 30d)

Description

  • Version: <= 10.13.0
  • Platform: Linux

when building with the --shared-openssl flag, it is highly probable that the openssl.cnf used by the shared library will be different than the one present in Node.js source code, resulting in several test failures. Typically, on debian, openssl is configured with stricter options regarding key length and TLS versions. It is very easy to avoid any test failure, though, by setting this environment variable when running tests: OPENSSL_CONF=./deps/openssl/openssl/apps/openssl.cnf

Guia do colaborador

Direção de pesquisa
O problema sugere definir a variável de ambiente OPENSSL CONF para o arquivo openssl.cnf incorporado ao executar testes com shared openssl. Isso pode ser implementado modificando a configuração do executor de testes, possivelmente no Makefile ou nos scripts de teste. Examine os arquivos de configuração de teste no repositório, como 'Makefile' ou 'test/common/', para adicionar a variável de ambiente.
Pilha de tecnologia
javascriptnodejs
Domain
backend
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.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitBuild toolsOpenSSL
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70