fix: MUST use GENESIS_FORK_VERSION to sign BLSToExecutionChange message.

from ethereum/consensus-specs#3206
lidofinance · Mar 10, 2023 · 1095d36 · 1095d36
1 parent b00af4c
commit 1095d36
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/pkg/wc_rotation/rotation.go b/pkg/wc_rotation/rotation.go
@@ -44,7 +44,7 @@ var (
 func GetSigningRoot(validatorIndex uint64) ([32]byte, error) {
 	domain, computeDomainErr := computeDomain(
 		DomainBlsToExecutionChange,
-		CapellaForkVersion,
+		GenesisForkVersion,
 		GenesisValidatorRoot,
 	)
 

diff --git a/pkg/wc_rotation/rotation_test.go b/pkg/wc_rotation/rotation_test.go
@@ -20,7 +20,7 @@ func TestGetSigningRoot(t *testing.T) {
 			args: args{
 				validatorIndex: 393395,
 			},
-			want:    `5c57b22ed4078f4e4e8ec3188c8c25895154a55c62525488f40b37a3464da6fe`,
+			want:    `23ccffc7767e1b9a54b3e18c986f00d0345825bcab21eae5fe92c849d6cfedb4`,
 			wantErr: false,
 		},
 	}
@@ -36,7 +36,7 @@ func TestGetSigningRoot(t *testing.T) {
 			actual := hex.EncodeToString(got[:])
 
 			if actual != tt.want {
-				t.Errorf("GetSigningRoot() got = %v, want %v", got, tt.want)
+				t.Errorf("GetSigningRoot() got = %v, want %v", actual, tt.want)
 			}
 		})
 	}