fix genesis_validators_root and missing @JsonCreator on BeaconStateMe…

…rge, ExecutionPayload and ExecutionPayloadHeader
Consensys · Nov 4, 2021 · 866d70a · 866d70a
1 parent 178c81d
commit 866d70a
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 0 deletions.
diff --git a/data/serializer/src/main/java/tech/pegasys/teku/api/schema/BeaconState.java b/data/serializer/src/main/java/tech/pegasys/teku/api/schema/BeaconState.java
@@ -35,6 +35,7 @@ public abstract class BeaconState implements State {
   @Schema(type = "string", format = "uint64")
   public final UInt64 genesis_time;
 
+  @Schema(type = "string", format = "byte", description = DESCRIPTION_BYTES32)
   public final Bytes32 genesis_validators_root;
 
   @Schema(type = "string", format = "uint64")

diff --git a/data/serializer/src/main/java/tech/pegasys/teku/api/schema/merge/BeaconStateMerge.java b/data/serializer/src/main/java/tech/pegasys/teku/api/schema/merge/BeaconStateMerge.java
@@ -13,6 +13,7 @@
 
 package tech.pegasys.teku.api.schema.merge;
 
+import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonProperty;
 import java.util.List;
 import java.util.Objects;
@@ -37,6 +38,7 @@ public class BeaconStateMerge extends BeaconStateAltair {
   @JsonProperty("latest_execution_payload_header")
   public final ExecutionPayloadHeader latestExecutionPayloadHeader;
 
+  @JsonCreator
   public BeaconStateMerge(
       @JsonProperty("genesis_time") final UInt64 genesisTime,
       @JsonProperty("genesis_validators_root") final Bytes32 genesisValidatorsRoot,

diff --git a/data/serializer/src/main/java/tech/pegasys/teku/api/schema/merge/ExecutionPayload.java b/data/serializer/src/main/java/tech/pegasys/teku/api/schema/merge/ExecutionPayload.java
@@ -13,6 +13,7 @@
 
 package tech.pegasys.teku.api.schema.merge;
 
+import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonProperty;
 import com.google.common.base.MoreObjects;
 import io.swagger.v3.oas.annotations.media.ArraySchema;
@@ -37,6 +38,7 @@ public class ExecutionPayload extends ExecutionPayloadCommon {
   @ArraySchema(schema = @Schema(type = "string", format = "byte"))
   public final List<Bytes> transactions;
 
+  @JsonCreator
   public ExecutionPayload(
       @JsonProperty("parent_hash") Bytes32 parentHash,
       @JsonProperty("coinbase") Bytes20 coinbase,

diff --git a/data/serializer/src/main/java/tech/pegasys/teku/api/schema/merge/ExecutionPayloadHeader.java b/data/serializer/src/main/java/tech/pegasys/teku/api/schema/merge/ExecutionPayloadHeader.java
@@ -15,6 +15,7 @@
 
 import static tech.pegasys.teku.api.schema.SchemaConstants.DESCRIPTION_BYTES32;
 
+import com.fasterxml.jackson.annotation.JsonCreator;
 import com.fasterxml.jackson.annotation.JsonProperty;
 import com.google.common.base.MoreObjects;
 import io.swagger.v3.oas.annotations.media.Schema;
@@ -37,6 +38,7 @@ public class ExecutionPayloadHeader extends ExecutionPayloadCommon {
       description = DESCRIPTION_BYTES32)
   public final Bytes32 transactionsRoot;
 
+  @JsonCreator
   public ExecutionPayloadHeader(
       @JsonProperty("parent_hash") Bytes32 parentHash,
       @JsonProperty("coinbase") Bytes20 coinbase,