Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

updated instructions for setting up a validator #9

Merged
merged 3 commits into from
Aug 16, 2024

Conversation

FidelVe
Copy link
Collaborator

@FidelVe FidelVe commented Aug 13, 2024

From feedback obtained by users in the discord channels I have updated the instructions for setting up and registering a validator to make it more easier for new people

Copy link

vercel bot commented Aug 13, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-icon-community ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 16, 2024 8:17pm

@eunsoo-icon
Copy link

For security reasons, it is recommended to separate the node and P-Rep keys.
P-Rep key is used to register a P-Rep and receive rewards.
node key is used for block consensus and is deployed on the node server. (./config/keystore.json)
A P-Rep must register the public key of its node key via setPRepNodePublicKey to vote on block consensus.

@FidelVe
Copy link
Collaborator Author

FidelVe commented Aug 14, 2024

For security reasons, it is recommended to separate the node and P-Rep keys. P-Rep key is used to register a P-Rep and receive rewards. node key is used for block consensus and is deployed on the node server. (./config/keystore.json) A P-Rep must register the public key of its node key via setPRepNodePublicKey to vote on block consensus.

Thank you for the feedback.

Ill rewrite it to reflect those recommendations.

@FidelVe
Copy link
Collaborator Author

FidelVe commented Aug 14, 2024

@eunsoo-icon

I just updated the instructions to add the recommendations of using one wallet for the validator and one wallet for the node.

please review it and let me know if is ok to publish

@FidelVe FidelVe merged commit 7f7fe14 into icon-community:master Aug 16, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants