microsoft / microsoft/BotFramework-DirectLineJS
DirectlineStreaming: Errors on disconnect are written to console
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 199
- Forks
- 133
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
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.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit dem Lesen von src/directLineStreaming.ts ungefähr bei den Zeilen 262 und 429 und untersuche anschließend das Ablehnungsverhalten von nodeWebSocketClient.ts. Überprüfe die anderen console.warn- und console.error-Aufrufe in der Bibliothek; die Aufgabe ist abgeschlossen, wenn Verbindungsabbaufehler und andere zutreffende Fehler über den Fehlerpfad der Anwendung weitergegeben werden, statt direkt in die Konsole geschrieben zu werden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- typescript
- Bereich
- api
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 48/100