peers/peerjs

Error expected on DataConnection when not establishing a connection

Open

#924 ouverte le 17 févr. 2022

Voir sur GitHub
 (5 commentaires) (7 réactions) (0 assignés)TypeScript (1 406 forks)batch import
clientfeaturehelp wanted

Métriques du dépôt

Stars
 (11 580 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Étudiez la classe DataConnection dans le code source (probablement `src/dataconnection.ts`). Comprenez comment la classe de base Peer émet des erreurs et reproduisez ce modèle dans DataConnection. Assurez vous que les erreurs lors de la configuration de la connexion (par exemple, ID invalide ou connexion à soi même) déclenchent le nouvel événement. Mettez à jour les typages TypeScript et écrivez des tests pour le nouvel événement.
Stack technique
typescript
Domaine
api
Type d'issue
Bug
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.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitNode.js
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60