rtfeldman/seamless-immutable

Incorrectly identifying promises

Open

#208 aperta il 18 mar 2017

Vedi su GitHub
 (6 commenti) (0 reazioni) (0 assegnatari)JavaScript (221 fork)batch import
bughelp wanted

Metriche repository

Star
 (5367 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

I've encountered an issue in the latest (7.1.0) release relating to commit 492d53e705acb6c

The issue is that isPromise makes a huge assumption that any object with a then method is a Promise. If the object is not a Promise, then will be called which may not be desirable.

This can be demonstrated very simply

Immutable({then:function() { console.log("should not be called") }});

I encountered this when using superagent which provides a thenable interface but is not technically a Promise. I suspect there may well be other libraries with a similar approach.

Guida contributor

Direzione di ricerca
Esamina la funzione isPromise nel codice sorgente. Attualmente controlla la presenza di un metodo 'then', identificando erroneamente oggetti thenable come Promise. Studia metodi affidabili per il rilevamento delle Promise, come l'uso di Promise.resolve() con cattura degli errori o il controllo di Symbol.toStringTag. Quindi proponi una soluzione per distinguere le Promise reali dai thenable.
Tech stack
javascript
Dominio
full stack
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70