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

[Polkadot WIki Migration] Set up a RPC node #32

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

CrackTheCode016
Copy link
Collaborator

infrastructure/general/setup-rpc.md Outdated Show resolved Hide resolved
infrastructure/general/setup-rpc.md Outdated Show resolved Hide resolved
infrastructure/general/setup-rpc.md Outdated Show resolved Hide resolved
infrastructure/general/setup-rpc.md Outdated Show resolved Hide resolved
infrastructure/general/setup-rpc.md Outdated Show resolved Hide resolved
infrastructure/general/setup-rpc.md Outdated Show resolved Hide resolved
infrastructure/general/setup-rpc.md Outdated Show resolved Hide resolved
@dawnkelly09
Copy link
Collaborator

Scan for Vale false positives. Resolved convos = made it on the update list and should go away now. I left comments for the legit flags with suggested edits. Thank you!

infrastructure/general/setup-rpc.md Outdated Show resolved Hide resolved
infrastructure/general/setup-rpc.md Outdated Show resolved Hide resolved
infrastructure/general/setup-rpc.md Outdated Show resolved Hide resolved
infrastructure/general/setup-rpc.md Outdated Show resolved Hide resolved
infrastructure/general/setup-rpc.md Outdated Show resolved Hide resolved
infrastructure/general/setup-rpc.md Outdated Show resolved Hide resolved
@dawnkelly09
Copy link
Collaborator

Alrighty, made a bunch of edits. Need a new content review from @0xLucca (especially pruned nodes part as I Googled it!) and format review from @eshaben

@dawnkelly09 dawnkelly09 requested review from eshaben and removed request for dawnkelly09 October 2, 2024 19:24
Copy link
Collaborator

@0xLucca 0xLucca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything looks fine. Just check the last comment I left

infrastructure/general/setup-rpc.md Show resolved Hide resolved
@@ -1,3 +1,4 @@
title: General Infrastructure
nav:
- index.md
- setup-rpc.md
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- setup-rpc.md
- 'Set Up an RPC Node': setup-rpc.md

@@ -0,0 +1,66 @@
---
title: Set Up an RPC Node
description: Learn how to securely configure, connect to, and maintain a Polkadot SDK-based RPC node in an archival or pruned state.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
description: Learn how to securely configure, connect to, and maintain a Polkadot SDK-based RPC node in an archival or pruned state.
description: Learn how to securely configure, connect to, and maintain an RPC node for a Polkadot SDK-based blockchain in an archival or pruned state.

This was a tad bit short

For an externally accessible Polkadot archive RPC node:

```config
polkadot --chain polkadot --name myrpc --state-pruning archive --blocks-pruning archive --rpc-max-connections 100 --rpc-cors all --rpc-methods Safe --rpc-port 9944
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
polkadot --chain polkadot --name myrpc --state-pruning archive --blocks-pruning archive --rpc-max-connections 100 --rpc-cors all --rpc-methods Safe --rpc-port 9944
polkadot --chain polkadot \
--name INSERT_YOUR_NODE_NAME \
--state-pruning archive \
--blocks-pruning archive \
--rpc-cors all \
--rpc-methods safe

I just removed the few that were set to default values

For a Polkadot pruned RPC node:

```config
polkadot --chain polkadot --name myrpc --state-pruning 1000 --blocks-pruning archive --rpc-max-connections 100 --rpc-cors all --rpc-methods Safe --rpc-port 9944
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
polkadot --chain polkadot --name myrpc --state-pruning 1000 --blocks-pruning archive --rpc-max-connections 100 --rpc-cors all --rpc-methods Safe --rpc-port 9944
polkadot --chain polkadot \
--name INSERT_YOUR_NODE_NAME \
--state-pruning 1000 \
--blocks-pruning archive \
--rpc-cors all \
--rpc-methods safe

Comment on lines +48 to +52
- You can limit the methods to use with `--rpc-methods`, an easy way to set this to a safe
mode is `--rpc-methods Safe`
- You can set your maximum connections through `--rpc-max-connections`, for example
`--rpc-max-connections 100`
- By default, localhost and Polkadot.js can access the RPC server. You can change this by setting `--rpc-cors`. To allow access from everywhere you can use `--rpc-cors all`
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- You can limit the methods to use with `--rpc-methods`, an easy way to set this to a safe
mode is `--rpc-methods Safe`
- You can set your maximum connections through `--rpc-max-connections`, for example
`--rpc-max-connections 100`
- By default, localhost and Polkadot.js can access the RPC server. You can change this by setting `--rpc-cors`. To allow access from everywhere you can use `--rpc-cors all`
- You can limit the methods to use with `--rpc-methods`; an easy way to set this to a safe mode is `--rpc-methods safe`
- You can set your maximum connections through `--rpc-max-connections`, for example, `--rpc-max-connections 200`
- By default, localhost and Polkadot.js can access the RPC server. You can change this by setting `--rpc-cors`. To allow access from everywhere, you can use `--rpc-cors all`

100 is the default so just changed it to another value

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants