microsoft / microsoft/BotFramework-DirectLineJS
DirectlineStreaming: Errors on disconnect are written to console
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- TypeScript
- Estrellas
- 199
- Forks
- 133
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
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.
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Empieza leyendo src/directLineStreaming.ts alrededor de las líneas 262 y 429; después, inspecciona el comportamiento de rechazo de nodeWebSocketClient.ts. Comprueba las demás llamadas a console.warn y console.error en la biblioteca; se considera terminado cuando los errores de desconexión y otros fallos aplicables se propagan a través de la ruta de errores de la aplicación en lugar de escribirse directamente en la consola.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- typescript
- Área
- api
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 48/100