Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

Splicing: pass candidate remote funding key to `hsmd_setup_channel`

オープン
#9,499 コメント 0 件 リアクション 0 件 担当者 1 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

@Andezion がすでに取り組んでいます。

2026年9月17日 から。

評価

難易度
3/5
見積もり時間
1〜2日
初心者へのやさしさ
68/100
issue の種類
バグ
明瞭さ
おおむね明確
活発さ
活発
技術スタック
c
領域
backend

調査の方向性

update_hsmd_with_splice() とその hsmd_setup_channel 呼び出しを追跡し、現在のチャネルの funding key と inflight->remote_funding を比較します。splicing 中に candidate key が渡されることを確認し、peer が funding key をローテーションする回帰テストを追加または実行して、signer が新しい funding outpoint とともに candidate key を受け取ることを確認します。

索引モデルが issue の本文から書いたものです。

説明

While working on VLS splicing integration, I came across what looks like a bug in the signer setup path when the peer rotates its funding key.

CLN correctly stores the candidate key in:

inflight->remote_funding

but update_hsmd_with_splice() still passes the current channel key to hsmd_setup_channel:

&peer->channel->funding_pubkey[REMOTE]

At that point the channel key is still the old funding key; it is only updated after splice_locked.

I suspect this was easy to miss because CLN currently does not rotate its own funding key during splicing, so CLN-to-CLN tests normally see the same key, and native hsmd does not validate the SetupChannel fields. For an external validating signer like VLS, however, this matters because it can receive the new funding outpoint together with the old remote funding key.

The fix seems to be to pass the candidate key instead:

&inflight->remote_funding

and if possible add a regression test where the peer actually rotates its funding key.

主要言語
C
スター
3.1k
フォーク
1k
平均マージ
4日 4時間
マージ済み PR(30日)
27

環境構築

  • Dockerfile または Docker Compose ファイルあり
  • プルリクエストのテンプレートあり
  • コントリビューションガイドなし

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

ElementsProject/lightning のほかの issue

ElementsProject/lightning の issue をすべて見る

似ている issue

C の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。