Nikita Kryuchkov

If the implementation of `go-ethereum` changes and uses `net.UDPConn`'s methods other than `ReadFromUDPAddrPort`, struct embedding will call `net.UDPConn`'s method directly and the `Unhandled` channel won't get drained. This PR would...

Closes https://github.com/ssvlabs/ssv/issues/1786

validator: listen and get metadata only for committee validators

5

Closes https://github.com/ssvlabs/ssv/issues/1703

Delete all pre-fork logic

``` --- FAIL: TestFetchLogsInBatches (0.31s) logger.go:146: 2024-10-11T19:29:35.699Z INFO execution_client connected to execution client {"address": "ws://127.0.0.1:35883", "took": "910.259µs"} --- FAIL: TestFetchLogsInBatches/startBlock_is_greater_than_endBlock (0.00s) execution_client_test.go:2[58](https://github.com/ssvlabs/ssv/actions/runs/11298162575/job/31426529860#step:5:59): Error Trace: /home/runner/work/ssv/ssv/eth/executionclient/execution_client_test.go:258 Error: Should not receive log...

we have many such errors on stage nodes ``` {"level":"\u001b[35mDEBUG\u001b[0m","time":"2024-10-11T18:47:48.016262Z","msg":"ignoring invalid message","duty_executor_id":"0000000000000000000000000000000086a7f656e5353ab6e594842f5203ebc0de276e1ed91f263af058f5ea9978a539","role":"COMMITTEE_RUNNER","ssv_message_type":"consensus","slot":2730838,"duty_id":"COMMITTEE-283_285_287_289-e85338-s2730838","round":5,"qbft_message_type":"round_change","peer_id":"16Uiu2HAmVLuJYsxWGdMRFLPL678aXLqTkXRa1irhnWzAAaQevHvi","error":"too many duties per epoch, got 33 (role COMMITTEE_RUNNER), want less than 32"} {"level":"\u001b[35mDEBUG\u001b[0m","time":"2024-10-11T18:47:48.014139Z","msg":"ignoring invalid message","duty_executor_id":"00000000000000000000000000000000c0bdc3d221e8bdc11d63fd6a91ac4e37fa690d8cf947a92fc39ba5a84ddd82a5","role":"COMMITTEE_RUNNER","ssv_message_type":"consensus","slot":2730838,"duty_id":"COMMITTEE-271_287_311_318-e85338-s2730838","round":5,"qbft_message_type":"round_change","peer_id":"16Uiu2HAmNTwEvEqrgVGyjyDuxEoPsPMgZXt5DzupPhaTnMeEFLYg","error":"too many duties per...

network/discovery: rate limit skipped peers log

2

@zktaiga reported that this log makes up to 40% of stage logs

message validation: rearrange checks

1

move ErrUnexpectedConsensusMessage ErrRoundTooHigh to be triggered as soon as possible to be able to replicate them in the attack sim @MatheusFranco99 please adjust the knowledge base

The current handling of records is very error-prone and causes bugs like https://github.com/ssvlabs/ssv/issues/1802. The code always expects subnets to be 128 byte long, so we can use `[128]byte` type instead...

Flaky TestBroadcaster

2

``` === RUN TestBroadcaster sent sent broadcaster_test.go:51: Error Trace: /Users/bloxmac/GolandProjects/ssv/exporter/api/broadcaster_test.go:51 Error: Not equal: expected: 0 actual : 1 Test: TestBroadcaster --- FAIL: TestBroadcaster (0.13s) Expected :0 Actual :1 ```