To-client sequence numbers are misleadingly logged as 0
Nadie ha tomado este issue todavía.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
I noticed this when debugging vscode-go, which inherits from vscode-debugadapter and relies on much of the code in LoggingDebugSession, etc.
Here is a stack trace for logging the following response:
To client: {"seq":0,"type":"response","request_seq":2,"command":"launch","success":true}
- ProtocolServer::_send # sets
message.seq = this._sequence++before sending - ProtocolServer::sendResponse
- LoggingDebugSession::SendResponse # logs the message with
"seq":0before calling the above functions
Same thing happens with events.
This is quite unfortunate because logging is an important for implementers who want to studying the interactions between the client and the adapter.
Is there any downside to moving logging after the raw send when the response object is populated with the right seq number?
- Lenguaje dominante
- TypeScript
- Estrellas
- 306
- Forks
- 90
- Merge medio
- 8 h 36 min
- PR fusionados (30 d)
- 3
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
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.
Más de microsoft/vscode-debugadapter-node
-
Dificultad 3/5 1-2 días Aptitud para principiantes 38/100
microsoft/vscode-debugadapter-node#325 · 1 comentario · 1 reacción ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
microsoft/vscode-debugadapter-node#307 · 1 comentario ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
Todos los issues de microsoft/vscode-debugadapter-node
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
area:tools bug good first issue help wanted priority:P2
Dificultad 2/5 1-3 horas Aptitud para principiantes 90/100
TaewoooPark/Motifcode#14 ·
-
bug 🐞
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
-
[Bounty proposal] fix(web): memory insights count an evening memory on the next day ($25 proposed) Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
BasedHardware/omi#15320 ·
-
Dificultad 2/5 Medio día Aptitud para principiantes 78/100
vercel/vercel-plugin#199 ·