fix(rln-relay): RLN DB should be aware of chain and contract address #670

richard-ramos · 2023-08-23T14:49:34Z

Description

See waku-org/nwaku#1921 for reasoning behind this fix

Adds chainID and contract address to RLN metadata
Adds test unit to verify metadata serialization
Returns an error during startup if metadata does not match configuration settings

part of #655

status-im-auto · 2023-08-23T14:51:08Z

❔	Commit	#️⃣	Finished (UTC)	Duration	Platform	Result
✔️	`24b049a`	#1	2023-08-23 14:51:05	~1 min	`linux`	📦`deb`
✖️	`24b049a`	#1	2023-08-23 14:51:52	~1 min	`tests`	📄`log`
✔️	`24b049a`	#1	2023-08-23 14:52:11	~2 min	`nix-flake`	📄`log`
✖️	`24b049a`	#1	2023-08-23 14:52:40	~2 min	`tests`	📄`log`
✔️	`24b049a`	#1	2023-08-23 14:54:00	~4 min	`android`	📦`tgz`
✔️	`24b049a`	#1	2023-08-23 14:54:36	~4 min	`ios`	📦`tgz`

✔️	`fa7ffb7`	#2	2023-08-23 16:08:01	~35 sec	`linux`	📦`deb`
✔️	`fa7ffb7`	#2	2023-08-23 16:09:29	~2 min	`nix-flake`	📄`log`
✔️	`fa7ffb7`	#2	2023-08-23 16:10:31	~3 min	`tests`	📄`log`
✔️	`fa7ffb7`	#2	2023-08-23 16:10:38	~3 min	`tests`	📄`log`
✔️	`fa7ffb7`	#2	2023-08-23 16:10:46	~3 min	`android`	📦`tgz`
✔️	`fa7ffb7`	#2	2023-08-23 16:12:38	~5 min	`ios`	📦`tgz`

rymnc · 2023-08-23T15:18:18Z

wow, this was really quick! Kudos!

chaitanyaprem

LGTM

richard-ramos added the E:2023-rln label Aug 23, 2023

fix(rln-relay): RLN DB should be aware of chain and contract address

fa7ffb7

richard-ramos force-pushed the fix/persist branch from 24b049a to fa7ffb7 Compare August 23, 2023 16:04

richard-ramos requested a review from chaitanyaprem August 23, 2023 17:45

chaitanyaprem approved these changes Aug 24, 2023

View reviewed changes

richard-ramos merged commit 5422af8 into master Aug 24, 2023
2 checks passed

richard-ramos deleted the fix/persist branch August 24, 2023 14:39