[Feature] Remove unused discovery message definitions
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 35/100
- issue の種類
- リファクタリング
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
- 技術スタック
- java
- 領域
- backend, networking
調査の方向性
Start with protocol/src/main/protos/core/Discover.proto and identify the four legacy message definitions; keep Endpoint and BackupMessage unchanged. Read the Chinese and English protocol documents and the compatibility discussion in issue #6921 before deciding whether removal can proceed. If approved, regenerate protobuf sources from a clean state, compile, run the handshake and backup keepalive regression tests, and compare retained-message serialization against pre-change fixtures.
索引モデルが issue の本文から書いたものです。
説明
Background
protocol/src/main/protos/core/Discover.proto still contains four legacy message definitions that are no longer referenced by java-tron’s runtime code: PingMessage, PongMessage, FindNeighbours, and Neighbours. These proto definitions are redundant leftovers from the libp2p split.
UDP discovery now uses the separate proto definitions in p2p/src/main/proto/Discover.proto (introduced in PR #6992).
Rationale
Remove unused definitions to reduce maintenance overhead and avoid confusion with the UDP discovery.
Specification
- Remove the four legacy messages from
protocol/src/main/protos/core/Discover.proto. - Preserve
Endpoint, referenced byHelloMessage.from, andBackupMessage, used by backup keepalive messages, with their existing fields and options. - Update the Chinese and English protocol documents to point to the real p2p discovery definitions.
Test Specification
- Regenerate protobuf sources from a clean state and compile the project.
- Run the related handshake and backup keepalive regression tests.
- Compare serialization against pre-change fixtures for
Endpoint,HelloMessage, andBackupMessage.
Scope of Impact
The cleanup is intended to preserve node network behavior and the wire encoding of retained messages. It removes generated types and changes the file descriptor, so downstream code referencing the removed types must migrate or retain a compatible protocol version. Projects using their own copies are unaffected unless they adopt the removal.
As clarified in #6921, these four generated message types are unsupported legacy types and do not require a deprecation cycle. Their removal intentionally changes the generated API and file descriptor, while preserving node interoperability and the wire encoding of the retained messages.
- 主要言語
- Java
- スター
- 4.2k
- フォーク
- 1.8k
- 平均マージ
- 3日 21時間
- マージ済み PR（30日）
- 13
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
tronprotocol/java-tron のほかの issue
-
[Feature] Validate ECKey inputs and improve key handling対応中かも @Federico2014 が 19 日前に担当しました。 オープンtype:feature
難易度 4/5 3〜5日 初心者へのやさしさ 40/100
tronprotocol/java-tron#6994 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
type:feature
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
tronprotocol/java-tron#6989 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
Robustness fixes for multiple issues対応中かも @xxo1shine が 21 日前に担当しました。 オープン
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
tronprotocol/java-tron#6969 · コメント 8 件 ·
メンテナーはふだん 1 日以内に返信
-
[Feature] decouple json-rpc filter processing from Manager対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンtype:feature
難易度 5/5 1週間以上 初心者へのやさしさ 48/100
tronprotocol/java-tron#6963 · コメント 8 件 ·
メンテナーはふだん 1 日以内に返信
-
[Feature] Remove unused SM2/SM3 crypto engine対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンtype:feature
難易度 5/5 1週間以上 初心者へのやさしさ 28/100
tronprotocol/java-tron#6959 · コメント 3 件 ·
メンテナーはふだん 1 日以内に返信
tronprotocol/java-tron の issue をすべて見る
似ている issue
-
needs-area-label
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
quelea-projection/Quelea#711 ·
-
bug pdd
難易度 2/5 1時間未満 初心者へのやさしさ 67/100
objectionary/eo#9450 ·
メンテナーはふだん 1 日以内に返信
-
[EC2] DescribeAvailabilityZones should return real AWS error for unknown zone name対応中かも @omatheusmesmo が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
メンテナーはふだん 1 日以内に返信
-
area: core good first issue type: enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100