nodejs/node

Run tests with OPENSSL_CONF set to embedded openssl.cnf

Open

#24.373 aperta il 15 nov 2018

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)JavaScript (35.535 fork)batch import
help wantedopensslstaletest

Metriche repository

Star
 (117.218 star)
Metriche merge PR
 (Merge medio 18g 17h) (219 PR mergiate in 30 g)

Descrizione

  • 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

Guida contributor

Direzione di ricerca
Il problema suggerisce di impostare la variabile d'ambiente OPENSSL CONF sul file openssl.cnf incorporato quando si eseguono i test con shared openssl. Ciò può essere implementato modificando la configurazione del test runner, possibilmente nel Makefile o negli script di test. Esamina i file di configurazione dei test nel repository, come 'Makefile' o 'test/common/', per aggiungere la variabile d'ambiente.
Tech stack
javascriptnodejs
Dominio
backend
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitBuild toolsOpenSSL
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70