To-client sequence numbers are misleadingly logged as 0
まだ誰も着手していません。
評価
この issue はまだ評価されていません。
説明
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?
- 主要言語
- TypeScript
- スター
- 306
- フォーク
- 90
- 平均マージ
- 8時間 36分
- マージ済み PR（30日）
- 3
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
microsoft/vscode-debugadapter-node のほかの issue
-
難易度 3/5 1〜2日 初心者へのやさしさ 38/100
microsoft/vscode-debugadapter-node#325 · コメント 1 件 · リアクション 1 件 ·
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 45/100
microsoft/vscode-debugadapter-node#307 · コメント 1 件 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 48/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
microsoft/vscode-debugadapter-node の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
bug v2
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
modelcontextprotocol/inspector#2458 · コメント 1 件 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 75/100
railmapgen/rmp-gallery#4068 ·
-
Mend: dependency security vulnerability status: needs triage 🕵️♀️
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
carbon-design-system/ibm-products#9907 ·