Skip to content

Commit

Permalink
Merge pull request eqlabs#2141 from eqlabs/krisztian/json-rpc-fix-rep…
Browse files Browse the repository at this point in the history
…laced-class-serialization

fix(rpc): fix serialization of `replaced_classes`
  • Loading branch information
kkovaacs authored Jul 29, 2024
2 parents 6a477ac + c8aabb3 commit 459f7d6
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

- Pathfinder does not properly limit the number of concurrent executors when using the `--rpc.execution-concurrency` CLI option.
- Pathfinder returns non-conforming `STRUCT_ABI_ENTRY` objects in response to `starknet_getClass` requests.
- Pathfinder returns `starknet_getStateUpdate` responses that are non-conformant with the specification if there are replaced classes in the response.

## [0.14.0] - 2024-07-22

Expand Down
5 changes: 1 addition & 4 deletions crates/rpc/src/dto/state_update.rs
Original file line number Diff line number Diff line change
Expand Up @@ -150,10 +150,7 @@ impl SerializeForVersion for StateDiff<'_> {
.filter_map(|(address, update)| {
update
.replaced_class()
.map(|class_hash| DeployedContractItem {
address,
class_hash,
})
.map(|hash| ReplacedClass { address, hash })
});

let mut nonces = self
Expand Down

0 comments on commit 459f7d6

Please sign in to comment.