Fork name decoding should be case insensitive #2583
Description
Description
I just checked in on my little Teku-Lighthouse Pyrmont validator for the first time in a while and found that it had failed to produce a few blocks with:
Sep 04 14:31:31.686 CRIT Error whilst producing block message: All endpoints failed https://eth2-beacon-pyrmont.infura.io/ => RequestFailed("Error from beacon node when producing block: Reqwest(reqwest::Error { kind: Decode, source: Error(\"Invalid fork name: ALTAIR\", line: 1, column: 19) })"), service: block
Specifically Teku BNs uppercase the fork name:
Invalid fork name: ALTAIR
This should be a one-line fix, a simple fork_name.lower() here:
lighthouse/consensus/types/src/fork_name.rs
Lines 61 to 65 in b0ac346
Plus a test.