alloy-rs / alloy-rs/alloy

[Bug] eip7840 BlobParams SerdeHelper drops two fields

Abierto
#3,103 1 comentario 0 reacciones 0 asignados Ver en GitHub
bug
Lenguaje dominante
Rust
Estrellas
1.3k
Forks
668
Merge medio
2 d 2 h
PR fusionados (30 d)
29

Descripción

### Component

serde

### What version of Alloy are you on?

1.0.41

### Operating System

None

### Describe the bug

Serde round-trip for a BlobParams instance is broken:
https://www.diffchecker.com/BcFU5LmK/

`max_blobs_per_tx` gets set to `max_blob_count`, while `blob_base_cost` is always set to zero because this behavior is hardcoded in the SerdeHelper:

https://github.com/alloy-rs/alloy/blob/c1987556cdcd0e99a1ce9c6512a8eea800df34fa/crates/eips/src/eip7840.rs#L219-L220

Guía de contribución

Abrir la guía de contribución

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.