peers/peerjs

Error expected on DataConnection when not establishing a connection

Open

#924 aperta il 17 feb 2022

Vedi su GitHub
 (5 commenti) (7 reazioni) (0 assegnatari)TypeScript (1406 fork)batch import
clientfeaturehelp wanted

Metriche repository

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

Descrizione

When a DataConnection fails to establish, for example, due to connecting to an invalid ID or to one's own ID, I would expect the dataConnection.on('error', () => {...}) callback to be executed. Instead, only peer.on('error', () => {}) catches such kinds of errors. This is especially annoying when you want to for example connect to multiple peers at once since we cannot know which one has failed to connect, making it hard to tell the user which peer(s) has issues.

Guida contributor

Direzione di ricerca
Analizza la classe DataConnection nel codice sorgente (probabilmente `src/dataconnection.ts`). Comprendi come la classe base Peer emette errori e replica quel pattern in DataConnection. Assicurati che gli errori durante la configurazione della connessione (ad esempio ID non valido o connessione a se stessi) attivino il nuovo evento. Aggiorna le definizioni dei tipi TypeScript e scrivi test per il nuovo evento.
Tech stack
typescript
Dominio
api
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
GitNode.js
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60