Skip to content

Commit

Permalink
feat(eigen-client-extra-features): Remove verify cert (#359)
Browse files Browse the repository at this point in the history
* Move inclusion logic

* Remove verify cert
  • Loading branch information
gianbelinche authored Nov 28, 2024
1 parent 01b50ed commit daace34
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 11 deletions.
1 change: 0 additions & 1 deletion core/node/da_clients/src/eigen/sdk.rs
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,6 @@ impl RawEigenClient {
.map_err(|e| anyhow::anyhow!("Failed to connect to Disperser server: {}", e))?;

let verifier_config = VerifierConfig {
verify_certs: true,
rpc_url: config.eigenda_eth_rpc.clone(),
svc_manager_addr: config.eigenda_svc_manager_address.clone(),
max_blob_size: config.blob_size_limit,
Expand Down
10 changes: 0 additions & 10 deletions core/node/da_clients/src/eigen/verifier.rs
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,6 @@ pub enum VerificationError {
/// Configuration for the verifier used for authenticated dispersals
#[derive(Debug, Clone)]
pub struct VerifierConfig {
pub verify_certs: bool,
pub rpc_url: String,
pub svc_manager_addr: String,
pub max_blob_size: u32,
Expand Down Expand Up @@ -466,9 +465,6 @@ impl Verifier {

/// Verifies that the certificate is valid
pub async fn verify_certificate(&self, cert: BlobInfo) -> Result<(), VerificationError> {
if !self.cfg.verify_certs {
return Ok(());
}
self.verify_batch(cert.clone()).await?;
self.verify_merkle_proof(cert.clone())?;
self.verify_security_params(cert.clone()).await?;
Expand All @@ -486,7 +482,6 @@ mod test {
#[test]
fn test_verify_commitment() {
let verifier = super::Verifier::new(super::VerifierConfig {
verify_certs: true,
rpc_url: "https://ethereum-holesky-rpc.publicnode.com".to_string(),
svc_manager_addr: "0xD4A7E1Bd8015057293f0D0A557088c286942e84b".to_string(),
max_blob_size: 2 * 1024 * 1024,
Expand Down Expand Up @@ -515,7 +510,6 @@ mod test {
#[test]
fn test_verify_merkle_proof() {
let verifier = super::Verifier::new(super::VerifierConfig {
verify_certs: true,
rpc_url: "https://ethereum-holesky-rpc.publicnode.com".to_string(),
svc_manager_addr: "0xD4A7E1Bd8015057293f0D0A557088c286942e84b".to_string(),
max_blob_size: 2 * 1024 * 1024,
Expand Down Expand Up @@ -605,7 +599,6 @@ mod test {
#[test]
fn test_hash_blob_header() {
let verifier = super::Verifier::new(super::VerifierConfig {
verify_certs: true,
rpc_url: "https://ethereum-holesky-rpc.publicnode.com".to_string(),
svc_manager_addr: "0xD4A7E1Bd8015057293f0D0A557088c286942e84b".to_string(),
max_blob_size: 2 * 1024 * 1024,
Expand Down Expand Up @@ -651,7 +644,6 @@ mod test {
#[test]
fn test_inclusion_proof() {
let verifier = super::Verifier::new(super::VerifierConfig {
verify_certs: true,
rpc_url: "https://ethereum-holesky-rpc.publicnode.com".to_string(),
svc_manager_addr: "0xD4A7E1Bd8015057293f0D0A557088c286942e84b".to_string(),
max_blob_size: 2 * 1024 * 1024,
Expand Down Expand Up @@ -680,7 +672,6 @@ mod test {
#[tokio::test]
async fn test_verify_batch() {
let verifier = super::Verifier::new(super::VerifierConfig {
verify_certs: true,
rpc_url: "https://ethereum-holesky-rpc.publicnode.com".to_string(),
svc_manager_addr: "0xD4A7E1Bd8015057293f0D0A557088c286942e84b".to_string(),
max_blob_size: 2 * 1024 * 1024,
Expand Down Expand Up @@ -770,7 +761,6 @@ mod test {
#[tokio::test]
async fn test_verify_security_params() {
let verifier = super::Verifier::new(super::VerifierConfig {
verify_certs: true,
rpc_url: "https://ethereum-holesky-rpc.publicnode.com".to_string(),
svc_manager_addr: "0xD4A7E1Bd8015057293f0D0A557088c286942e84b".to_string(),
max_blob_size: 2 * 1024 * 1024,
Expand Down

0 comments on commit daace34

Please sign in to comment.