fix(rln-relay): keystore usage #1750
Conversation
| Otherwise, the chat client does not generate a new credential and will use, instead, the persisted RLN credential. | ||
|
|
||
| ```bash | ||
| ./build/chat2 --fleet:test --content-topic:/toy-chat/3/mingde/proto --rln-relay:true --rln-relay-dynamic:true --rln-relay-eth-contract-address:0x9C09146844C1326c2dBC41c451766C7138F88155 --rln-relay-eth-account-private-key:your_eth_private_key --rln-relay-eth-client-address:your_sepolia_node --ports-shift=1 --rln-relay-cred-path:./ | ||
| ./build/chat2 --fleet:test --content-topic:/toy-chat/3/mingde/proto --rln-relay:true --rln-relay-dynamic:true --rln-relay-eth-contract-address:0x9C09146844C1326c2dBC41c451766C7138F88155 --rln-relay-eth-account-private-key:your_private_key --rln-relay-eth-client-address:your_sepolia_node --ports-shift=1 --rln-relay-cred-path:./rlnKeystore.json --rln-relay-cred-password:your_password |
There was a problem hiding this comment.
I understood from the description above that the --rln-relay-cred-path is supposed to be a path to folder - since it explicitly states that is would persist the cred into rlnKeystore.json file under the given path, but the example specifies full path including the file
So it seems either the doc or the example is not fully correct.
There was a problem hiding this comment.
Hmm, the given command works. Probably the doc doesn't explicitly mention that you have to pass a filename in. I'll make the change.
There was a problem hiding this comment.
Is this better? f0c7fcb
There was a problem hiding this comment.
Although not being requested but it lgtm :)
The tutorial is super well explained!
Maybe it worth adding a reference about how to get eth into the sepolia account? i.e. https://sepoliafaucet.com/ or at least mentioning that you can get eth from there.
Description
Update operator docs to include keystore change for rln-relay
Changes
Issue
Part of #1624