Anton Nashatyrev

Add networking req/resp request rate limiting rule

11

The networking spec is still missing any request rate limit rules. That leads to the situation when different clients make their own assumptions on 'reasonable' rate limits and could disconnect...

[Sharding] Forkchoice: compute DAS data dependencies starting from the finalized block

6

According to the present spec a block is eligible for consideration only when _all_ previous blocks (starting from the sharding fork) pass DAS verification. Consider a node syncing from scratch...

fix the problem when Change.getRemoved() returns other mapped instances than get()

1

That issue breaks JFX charts with current `MappedList` implementation. (BTW nice to meet you again ;)

Bytes.fromByteBuf() incorrectly interpret ByteBuf content

8

### To reproduce ```java ByteBuf buf = Unpooled.buffer(100).writeByte(1); Bytes bytes = Bytes.wrapByteBuf(buf); System.out.println(bytes.size()); System.out.println(bytes.toArray().length); ``` ### Expected behavior ``` 1 1 ``` The `buf` content is only 1 byte, other...

Explicitly validate public key when processing Deposit

1

Deposits are the only source of non-validated public keys, so suggesting to add explicit public key validation statement to `process_deposit` function. `bls.Verify` function (and its batch variants) is a bit...

Regression from fix: older GPUs show x2-x3 slowdown

1

https://github.com/KlausT/ccminer-cryptonight/commit/824db322bb7f0dad920711ca29dbe1457ff79f49 This fix makes some older (or just some mobile only) GPU perform 2x-3x slower with default settings. Reproduced on 840M and 960M 840M (3 smx): - before the fix:...

Currently `ObservableStateProcessorImpl.drainAttestations` drains all received attestations just by the state epoch. But those attestations can be from the other fork and calculation of validator indices may not make any sense.

Some further SSZ work

1

As the follow up to PR #115 I would like to do the following things: - ~~add Incremental hashing of basic lists (packed)~~ - refactor `SSZBasicAccessors` - make `ssz` module...

See: `org.ethereum.beacon.crypto.Hashes#digestUsingAlgorithm` The default java `Provider.Service` class instantiates `Digest` using reflection which causes 10-15% of performance degradation (it looks up the class constructor on each instantiation). The `Service` class javadoc...

Redesign ObservableStateProcessor

1

We need probably split this service onto separate loosely coupled services like: - HeadTracker - PendingOperationsTracker - ObservableStateProcessor `PendingOperationsTracker` should correctly handle head rebranching `ObservableStateProcessor` should probably expose 2 streams...