peers/peerjs

Error expected on DataConnection when not establishing a connection

Open

#924 aberto em 17 de fev. de 2022

Ver no GitHub
 (5 comments) (7 reactions) (0 assignees)TypeScript (1.406 forks)batch import
clientfeaturehelp wanted

Métricas do repositório

Stars
 (11.580 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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.

Guia do colaborador

Direção de pesquisa
Investigue a classe DataConnection no código fonte (provavelmente em `src/dataconnection.ts`). Entenda como a classe base Peer emite erros e replique esse padrão em DataConnection. Garanta que erros durante a configuração da conexão (por exemplo, ID inválido ou conexão consigo mesmo) acionem o novo evento. Atualize as definições de tipo do TypeScript e escreva testes para o novo evento.
Pilha de tecnologia
typescript
Domain
api
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitNode.js
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60