nodejs/node

Run tests with OPENSSL_CONF set to embedded openssl.cnf

Open

#24 373 ouverte le 15 nov. 2018

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)JavaScript (35 535 forks)batch import
help wantedopensslstaletest

Métriques du dépôt

Stars
 (117 218 stars)
Métriques de merge PR
 (Merge moyen 18j 17h) (219 PRs mergées en 30 j)

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

Guide contributeur

Direction de recherche
Le problème suggère de définir la variable d'environnement OPENSSL CONF sur le fichier openssl.cnf intégré lors de l'exécution des tests avec shared openssl. Cela peut être implémenté en modifiant la configuration du lanceur de tests, éventuellement dans le Makefile ou les scripts de test. Examinez les fichiers de configuration de test dans le dépôt, tels que 'Makefile' ou 'test/common/', pour ajouter la variable d'environnement.
Stack technique
javascriptnodejs
Domaine
backend
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitBuild toolsOpenSSL
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70