lambdaclass / ethrex

ethrex: Ethereum Rust Execution L1 and L2 client.
Apache License 2.0
238 stars 23 forks source link

L1: Add a struct to handle RLPx connections context #1140

Open fkrause98 opened 2 weeks ago

fkrause98 commented 2 weeks ago

Issue

Currently, we're passing 4-5 arguments to the functions under p2p/net.rs, let's consider adding a structure that contains these arguments as fields to avoid it becoming hard to read and maintain.

varun-doshi commented 4 days ago

Checked out this file...there are a few different fields in the functions Would you like the struct to have all the fields with most of them being an Option<...> type This struct would get passed directly to the functions with the fields as per required by respective functions