microsoft / microsoft/BotFramework-DirectLineJS
DirectlineStreaming: Errors on disconnect are written to console
Personne n'a encore pris cette issue.
- Langage dominant
- TypeScript
- Étoiles
- 199
- Forks
- 133
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
When disconnecting a connection that has pending requests, the infra botframework-streaming package rejects all the requests with "Disconnect was called" error.
It looks like the botframework-directlinejs package writes this error to the console instead/in addition to emitting an error event to the application.
This bypasses our logging system, and should not be done by libraries.
I suspect the console log comes from here or from here.
There are other calls to console.warn and console.error in the library. They should be replaced where possible with proper error propagation.
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par lire src/directLineStreaming.ts autour des lignes 262 et 429, puis examinez le comportement de rejet de nodeWebSocketClient.ts. Vérifiez les autres appels à console.warn et console.error dans la bibliothèque ; le travail est terminé lorsque les erreurs de déconnexion et les autres échecs applicables sont propagés via le chemin de gestion des erreurs de l’application au lieu d’être écrits directement dans la console.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- typescript
- Domaine
- api
- Type d'issue
- Bug
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 48/100