nodejs/undici

Filter tls options

Open

Aperta il 28 giu 2020

Vedi su GitHub
 (6 commenti) (0 reazioni) (0 assegnatari)JavaScript (5319 star) (444 fork)batch import
enhancementgood first issue

Descrizione

Determine which tls options actually make sense to forward to tls.connect and throw if any other is provided (e.g. allowHalfOpen).

Guida contributor

Tech stack
javascriptnodejs
Dominio
backendapi
Tipo issue
feature
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
half day
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
fresh
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
Node.js basicsTLS conceptsUnderstanding of undici codebase
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70
Direzione di ricerca
Examine the undici source code to find where TLS options are currently passed to `tls.connect`. Likely in a connection file (e.g., `lib/connect.js` or similar). Review Node.js documentation for `tls.connect` to list valid options. Then implement a filter that allows only valid TLS options and throws an error for invalid ones like `allowHalfOpen`. Consider adding tests for the new validation.