Skip to content

feat(l1): implement discv5 nodes message coding#5630

Merged
edg-l merged 2 commits into
discv5from
discv5_nodes_coding
Dec 15, 2025
Merged

feat(l1): implement discv5 nodes message coding#5630
edg-l merged 2 commits into
discv5from
discv5_nodes_coding

Conversation

@edg-l

@edg-l edg-l commented Dec 15, 2025

Copy link
Copy Markdown
Contributor

Checklist

  • Updated STORE_SCHEMA_VERSION (crates/storage/lib.rs) if the PR includes breaking changes to the Store requiring a re-sync.

Closes #5578 and #5579

@edg-l edg-l requested a review from a team as a code owner December 15, 2025 08:32
@edg-l edg-l changed the title Add discv5 nodes message encoding decoding feat(l1): implement discv5 nodes message coding Dec 15, 2025
@github-actions github-actions Bot added the L1 Ethereum client label Dec 15, 2025
@github-actions

github-actions Bot commented Dec 15, 2025 •
edited
Loading

Copy link
Copy Markdown

Lines of code report

Total lines added: 59
Total lines removed: 0
Total lines changed: 59

Detailed view 
+-------------------------------------------------+-------+------+
| File                                            | Lines | Diff |
+-------------------------------------------------+-------+------+
| ethrex/crates/networking/p2p/discv5/messages.rs | 481   | +59  |
+-------------------------------------------------+-------+------+

@github-project-automation github-project-automation Bot moved this to In Review in ethrex_l1 Dec 15, 2025
@edg-l edg-l force-pushed the discv5_nodes_coding branch from 7504de3 to 22dbd21 Compare December 15, 2025 12:18
@edg-l edg-l merged commit 552160a into discv5 Dec 15, 2025
21 of 22 checks passed
@edg-l edg-l deleted the discv5_nodes_coding branch December 15, 2025 12:52
@github-project-automation github-project-automation Bot moved this from In Review to Done in ethrex_l1 Dec 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ElFantasma ElFantasma ElFantasma approved these changes

+1 more reviewer

@azteca1998 azteca1998 azteca1998 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@edg-l edg-l

Labels

L1 Ethereum client

Projects

ethrex_l1
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@edg-l @azteca1998 @ElFantasma