Refactor: Consider removing usage of setTimeout

Aperta
#894 0 commenti 2 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
25/100
Tipo di issue
Refactoring
Chiarezza
Da chiarire
Stato di attività
Ferma
Stack tecnologico
javascript
Ambito
backend

Direzione di ricerca

Inizia leggendo il commit 798 della pull request collegata e individuando l’uso di setTimeout che modifica. Segui il callback close correlato e il comportamento della race-condition, quindi esamina la configurazione dei test esistente; il lavoro è completato quando l’uso del timeout viene rimosso o la sua necessità viene risolta usando l’evento close proposto, senza introdurre regressioni nell’analisi multipart.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Area: files Type: Enhancement
What problem are you trying to solve?

https://github.com/node-formidable/formidable/pull/798/commits/a11190f7e4e40de8f799cece87e56a59dc63164f

Maybe do not use setTimeout

Do you have a new or modified API suggestion to solve the problem?

Use on('close' callback.

Other

Not sure, this is race condition specific, so traditional unit tests don't work

Lingua principale
JavaScript
Stelle
7.2k
Fork
689
Merge medio
3g 7h
PR unite (30g)
2

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di node-formidable/formidable

Tutte le issue di node-formidable/formidable

Issue simili

Altre issue su JavaScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.