DVC: `NOW_RDM_CAPABILITIES` error path sends no response
メンテナーはふだん 1 日以内に返信
評価
調査の方向性
Start at the error arm of NowRdmMessage::Capabilities in devolutions-session/src/dvc/task.rs (around lines 625-633), then trace the Err causes in rdm.rs (process_capabilities, the pre-epoch clock check, and the response construction). Decide whether the failure can be expressed as an app-available-unset response or must close the channel, then add the test CapabilitiesProcessingFailureRespondsOrClosesChannel. Linked PR #2096 is already open against this issue, so check it before starting.
索引モデルが issue の本文から書いたものです。
説明
Where: devolutions-session/src/dvc/task.rs:625-633 (error arm of NowRdmMessage::Capabilities); causes at rdm.rs:378-381 (clock before the Unix epoch), rdm.rs:405-407 (response construction).
What happens: when process_capabilities returns Err, the arm logs and sends nothing. The happy path - including "RDM not installed" (rdm.rs:388-416) - always answers with app-available unset, so this only triggers on genuinely odd inputs. The consequence is still permanent: the client's send-order matching is shifted for the rest of the connection.
Ask: always answer or close. If the failure can be represented in the response, respond with app-available unset; otherwise close the channel with an error. Don't leave a usable-looking connection missing a required response.
Test: CapabilitiesProcessingFailureRespondsOrClosesChannel.
Context: Devolutions/now-libraries#115.
- 主要言語
- Rust
- スター
- 162
- フォーク
- 30
- 平均マージ
- 1日 9時間
- マージ済み PR（30日）
- 60
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
Devolutions/devolutions-gateway のほかの issue
-
test(testsuite): drop the devolutions-gateway dev-dependency対応中かも @CBenoit が 1 日前に担当しました。 オープン
難易度 4/5 3〜5日 初心者へのやさしさ 18/100
Devolutions/devolutions-gateway#2090 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
Devolutions/devolutions-gateway#2047 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
Devolutions/devolutions-gateway#1973 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
Devolutions/devolutions-gateway#1930 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 55/100
Devolutions/devolutions-gateway#1788 ·
メンテナーはふだん 1 日以内に返信
Devolutions/devolutions-gateway の issue をすべて見る
似ている issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
pop-os/cosmic-comp#2956 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
libp2p/rust-libp2p#6694 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
EmbarkStudios/cargo-deny#910 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
tursodatabase/turso#10054 ·
メンテナーはふだん 1 日以内に返信
-
build: Update rust version対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンdependencies
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
ibm-granite-community/granite-cli#229 ·
メンテナーはふだん 1 日以内に返信