Skip to content

Commit 4ab5131

Browse files
committed
Upgrade rust-eth-kzg to 0.8.0.
1 parent 6604fd1 commit 4ab5131

File tree

13 files changed

+182
-188
lines changed

13 files changed

+182
-188
lines changed

Cargo.lock

Lines changed: 122 additions & 57 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -221,7 +221,7 @@ reqwest = { version = "0.11", default-features = false, features = [
221221
ring = "0.17"
222222
rpds = "0.11"
223223
rusqlite = { version = "0.28", features = ["bundled"] }
224-
rust_eth_kzg = "0.5.4"
224+
rust_eth_kzg = "0.8.0"
225225
safe_arith = { path = "consensus/safe_arith" }
226226
sensitive_url = { path = "common/sensitive_url" }
227227
serde = { version = "1", features = ["derive"] }

beacon_node/beacon_chain/src/kzg_utils.rs

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -423,7 +423,7 @@ mod test {
423423
use bls::Signature;
424424
use eth2::types::BlobsBundle;
425425
use execution_layer::test_utils::generate_blobs;
426-
use kzg::{trusted_setup::get_trusted_setup, Kzg, KzgCommitment, TrustedSetup};
426+
use kzg::{trusted_setup::get_trusted_setup, Kzg, KzgCommitment};
427427
use types::{
428428
beacon_block_body::KzgCommitments, BeaconBlock, BeaconBlockFulu, BlobsList, ChainSpec,
429429
EmptyBlock, EthSpec, ForkName, FullPayload, KzgProofs, MainnetEthSpec, SignedBeaconBlock,
@@ -568,10 +568,7 @@ mod test {
568568
}
569569

570570
fn get_kzg() -> Kzg {
571-
let trusted_setup: TrustedSetup = serde_json::from_reader(get_trusted_setup().as_slice())
572-
.map_err(|e| format!("Unable to read trusted setup file: {}", e))
573-
.expect("should have trusted setup");
574-
Kzg::new_from_trusted_setup_das_enabled(trusted_setup).expect("should create kzg")
571+
Kzg::new_from_trusted_setup(&get_trusted_setup()).expect("should create kzg")
575572
}
576573

577574
fn create_test_fulu_block_and_blobs<E: EthSpec>(

beacon_node/beacon_chain/src/test_utils.rs

Lines changed: 4 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ use futures::channel::mpsc::Receiver;
3333
pub use genesis::{InteropGenesisBuilder, DEFAULT_ETH1_BLOCK_HASH};
3434
use int_to_bytes::int_to_bytes32;
3535
use kzg::trusted_setup::get_trusted_setup;
36-
use kzg::{Kzg, TrustedSetup};
36+
use kzg::Kzg;
3737
use logging::create_test_tracing_subscriber;
3838
use merkle_proof::MerkleTree;
3939
use operation_pool::ReceivedPreCapella;
@@ -81,33 +81,18 @@ pub const TEST_DATA_COLUMN_SIDECARS_SSZ: &[u8] =
8181
pub const DEFAULT_TARGET_AGGREGATORS: u64 = u64::MAX;
8282

8383
static KZG: LazyLock<Arc<Kzg>> = LazyLock::new(|| {
84-
let trusted_setup: TrustedSetup = serde_json::from_reader(get_trusted_setup().as_slice())
85-
.map_err(|e| format!("Unable to read trusted setup file: {}", e))
86-
.expect("should have trusted setup");
87-
let kzg = Kzg::new_from_trusted_setup(trusted_setup).expect("should create kzg");
88-
Arc::new(kzg)
89-
});
90-
91-
static KZG_PEERDAS: LazyLock<Arc<Kzg>> = LazyLock::new(|| {
92-
let trusted_setup: TrustedSetup = serde_json::from_reader(get_trusted_setup().as_slice())
93-
.map_err(|e| format!("Unable to read trusted setup file: {}", e))
94-
.expect("should have trusted setup");
95-
let kzg = Kzg::new_from_trusted_setup_das_enabled(trusted_setup).expect("should create kzg");
84+
let kzg = Kzg::new_from_trusted_setup(&get_trusted_setup()).expect("should create kzg");
9685
Arc::new(kzg)
9786
});
9887

9988
static KZG_NO_PRECOMP: LazyLock<Arc<Kzg>> = LazyLock::new(|| {
100-
let trusted_setup: TrustedSetup = serde_json::from_reader(get_trusted_setup().as_slice())
101-
.map_err(|e| format!("Unable to read trusted setup file: {}", e))
102-
.expect("should have trusted setup");
103-
let kzg = Kzg::new_from_trusted_setup_no_precomp(trusted_setup).expect("should create kzg");
89+
let kzg =
90+
Kzg::new_from_trusted_setup_no_precomp(&get_trusted_setup()).expect("should create kzg");
10491
Arc::new(kzg)
10592
});
10693

10794
pub fn get_kzg(spec: &ChainSpec) -> Arc<Kzg> {
10895
if spec.fulu_fork_epoch.is_some() {
109-
KZG_PEERDAS.clone()
110-
} else if spec.deneb_fork_epoch.is_some() {
11196
KZG.clone()
11297
} else {
11398
KZG_NO_PRECOMP.clone()

beacon_node/client/src/builder.rs

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -185,13 +185,10 @@ where
185185
};
186186

187187
let kzg_err_msg = |e| format!("Failed to load trusted setup: {:?}", e);
188-
let trusted_setup = config.trusted_setup.clone();
189188
let kzg = if spec.is_peer_das_scheduled() {
190-
Kzg::new_from_trusted_setup_das_enabled(trusted_setup).map_err(kzg_err_msg)?
191-
} else if spec.deneb_fork_epoch.is_some() {
192-
Kzg::new_from_trusted_setup(trusted_setup).map_err(kzg_err_msg)?
189+
Kzg::new_from_trusted_setup(&config.trusted_setup).map_err(kzg_err_msg)?
193190
} else {
194-
Kzg::new_from_trusted_setup_no_precomp(trusted_setup).map_err(kzg_err_msg)?
191+
Kzg::new_from_trusted_setup_no_precomp(&config.trusted_setup).map_err(kzg_err_msg)?
195192
};
196193

197194
let builder = BeaconChainBuilder::new(eth_spec_instance, Arc::new(kzg))

beacon_node/client/src/config.rs

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
use beacon_chain::graffiti_calculator::GraffitiOrigin;
22
use beacon_chain::validator_monitor::ValidatorMonitorConfig;
3-
use beacon_chain::TrustedSetup;
43
use beacon_processor::BeaconProcessorConfig;
54
use directory::DEFAULT_ROOT_DIR;
65
use environment::LoggerConfig;
@@ -70,7 +69,7 @@ pub struct Config {
7069
pub network: network::NetworkConfig,
7170
pub chain: beacon_chain::ChainConfig,
7271
pub execution_layer: Option<execution_layer::Config>,
73-
pub trusted_setup: TrustedSetup,
72+
pub trusted_setup: Vec<u8>,
7473
pub http_api: http_api::Config,
7574
pub http_metrics: http_metrics::Config,
7675
pub monitoring_api: Option<monitoring_api::Config>,
@@ -84,9 +83,6 @@ pub struct Config {
8483

8584
impl Default for Config {
8685
fn default() -> Self {
87-
let trusted_setup: TrustedSetup = serde_json::from_reader(get_trusted_setup().as_slice())
88-
.expect("Unable to read trusted setup file");
89-
9086
Self {
9187
data_dir: PathBuf::from(DEFAULT_ROOT_DIR),
9288
db_name: "chain_db".to_string(),
@@ -98,7 +94,7 @@ impl Default for Config {
9894
network: NetworkConfig::default(),
9995
chain: <_>::default(),
10096
execution_layer: None,
101-
trusted_setup,
97+
trusted_setup: get_trusted_setup(),
10298
beacon_graffiti: GraffitiOrigin::default(),
10399
http_api: <_>::default(),
104100
http_metrics: <_>::default(),

beacon_node/execution_layer/src/test_utils/execution_block_generator.rs

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -937,7 +937,7 @@ pub fn generate_pow_block(
937937
#[cfg(test)]
938938
mod test {
939939
use super::*;
940-
use kzg::{trusted_setup::get_trusted_setup, Bytes48, CellRef, KzgBlobRef, TrustedSetup};
940+
use kzg::{trusted_setup::get_trusted_setup, Bytes48, CellRef, KzgBlobRef};
941941
use types::{MainnetEthSpec, MinimalEthSpec};
942942

943943
#[test]
@@ -1056,10 +1056,7 @@ mod test {
10561056
}
10571057

10581058
fn load_kzg() -> Result<Kzg, String> {
1059-
let trusted_setup: TrustedSetup =
1060-
serde_json::from_reader(get_trusted_setup().as_slice())
1061-
.map_err(|e| format!("Unable to read trusted setup file: {e:?}"))?;
1062-
Kzg::new_from_trusted_setup(trusted_setup)
1059+
Kzg::new_from_trusted_setup(&get_trusted_setup())
10631060
.map_err(|e| format!("Failed to load trusted setup: {e:?}"))
10641061
}
10651062
}

0 commit comments

Comments
 (0)