`exit verify` check Capella fork version

[imnisen]

Hi, does the ethdo exit verify cmd need to upgrade to support EIP-7044, code staring from

ethdo/cmd/exitverify.go

Lines 86 to 101 in 25a5bd9

	// Try with the current fork.
	domain := phase0.Domain{}
	currentExitDomain, err := e2types.ComputeDomain(e2types.DomainVoluntaryExit, response.Data.CurrentVersion[:], genesis.GenesisValidatorsRoot[:])
	errCheck(err, "Failed to compute domain")
	copy(domain[:], currentExitDomain)
	verified, err = util.VerifyRoot(account, opRoot, domain, sig)
	errCheck(err, "Failed to verify voluntary exit")
	if !verified {
	// Try again with the previous fork.
	previousExitDomain, err := e2types.ComputeDomain(e2types.DomainVoluntaryExit, response.Data.PreviousVersion[:], genesis.GenesisValidatorsRoot[:])
	copy(domain[:], previousExitDomain)
	errCheck(err, "Failed to compute domain")
	verified, err = util.VerifyRoot(account, opRoot, domain, sig)
	errCheck(err, "Failed to verify voluntary exit")
	}
	assert(verified, "Voluntary exit failed to verify against current and previous fork versions")

It might not works after Pectra upgrade because it checks the current and the previous fork only.

[aimxhaisse]

Yup, it's broken since Dencun with EIP-7044:

Modify process_voluntary_exit to compute the signing domain and root fixed on CAPELLA_FORK_VERSION.

From spec:

    domain = compute_domain(DOMAIN_VOLUNTARY_EXIT, CAPELLA_FORK_VERSION, state.genesis_validators_root)
    signing_root = compute_signing_root(voluntary_exit, domain)

Domain's fork should be fixed to CAPELLA_FORK_VERSION, instead of getting the latest/previous fork version from the beacon.

[aimxhaisse]

Tentative fix @ #165