onbjerg

Yes, you need to mark the fields as public. Please see https://doc.rust-lang.org/rust-by-example/mod/struct_visibility.html

@guha-rahul Yes, don't use `CLIENTVERSIONV1` in the tests for the method, just fill it with dummy fields, which is fine for testing. This should get rid of the cyclical dependency

@joshieDo please link the PR you have open for this

This is because you can't have more than one threadpool per process, I ran into this with one of the benchmarks we hadn't kept up to date, too. Not entirely...

Yeah reading the threadpoolbuilder docs, it returns an error but does not mention anything about it being fatal or stalling, so should be ok: https://docs.rs/rayon/latest/rayon/struct.ThreadPoolBuilder.html#method.build_global

Hey @allnil, what's the status of this?

Can you expand a bit on how you intend to solve it? And are there any unknowns for you/things you need clarified?

@int88 sorry for the delayed response, I think what OP wants is not just ipv6 support like that, it's also the ability to specify `--nat` multiple times, once with an...

Unfamiliar with how extip is supposed to work in the ENR, I don't know if we can advertise an ipv4 and an ipv6 address at the same time, cc @mattsse

What node version?