waku-org / nwaku

Waku node and protocol.

Other

183 stars 46 forks source link

test: remove some API message validation checks #2660

Open stubbsta opened 2 weeks ago

stubbsta commented 2 weeks ago

Description

DO NOT MERGE, only for RLNv2 epoch testing

Changes

The following message validation checks are commented out:

nonceLimit check in nonce_manager.nim
message validation in handlers.nim

How to test

This procedure can be used for testing: https://www.notion.so/Waku-RLNv2-Testnet1-d11b3108a6ed4c39b378417db82302a6

Issue

Relates to https://github.com/waku-org/waku-simulator/issues/44

github-actions[bot] commented 2 weeks ago

You can find the image built from this PR at

quay.io/wakuorg/nwaku-pr:2660-rln-v2

Built from ea2344044d3d27e670c8b14a3cdb11ea431f35a3

github-actions[bot] commented 2 weeks ago

You can find the image built from this PR at

quay.io/wakuorg/nwaku-pr:2660-rln-v1

Built from ea2344044d3d27e670c8b14a3cdb11ea431f35a3

alrevuelta commented 2 weeks ago

Looks good. Not meant to be merged but we built this to have an nwaku node that allows to inject messages exceeding the rate limit. Without the commented checks, the node will skip RLN validation and will publish no matter what.

By now we are using the image quay is building and at some point we will switch to waku-spameerrrr