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

[Merged by Bors] - Add community checkpoint sync endpoints to book #3558

Closed
wants to merge 1 commit into from

Conversation

remyroy
Copy link
Contributor

@remyroy remyroy commented Sep 9, 2022

Proposed Changes

Add a section on the new community checkpoint sync endpoints in the book. This should help stakers sync faster even without having to create an account.

Copy link
Member

@michaelsproul michaelsproul left a comment

Choose a reason for hiding this comment

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

Niiice!

@michaelsproul michaelsproul added docs Documentation ready-for-merge This PR is ready to merge. labels Sep 9, 2022
@michaelsproul
Copy link
Member

bors r+

bors bot pushed a commit that referenced this pull request Sep 9, 2022
## Proposed Changes

Add a section on the new community checkpoint sync endpoints in the book. This should help stakers sync faster even without having to create an account.
@coincashew
Copy link

Just a suggestion. The / has caused issues for some CL clients.

recommend
--checkpoint-sync-url=https://beaconstate.info

@bors bors bot changed the title Add community checkpoint sync endpoints to book [Merged by Bors] - Add community checkpoint sync endpoints to book Sep 9, 2022
@bors bors bot closed this Sep 9, 2022

The Ethereum community provides various [public endpoints](https://eth-clients.github.io/checkpoint-sync-endpoints/) for you to choose from for your initial checkpoint state. Select one for your network and use it as the url for the `--checkpoint-sync-url` flag. e.g.
```
lighthouse bn --checkpoint-sync-url https://beaconstate.info/ ...
Copy link

@InvisibleSymbol InvisibleSymbol Sep 9, 2022

Choose a reason for hiding this comment

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

I feel like People might just end up copying this line out of laziness and all end up using this specific endpoint, which is sub-optimal from a safety perspective (no single checkpoint endpoint should be recommended). How about using a non-working domain instead, like example.com?

Copy link
Member

Choose a reason for hiding this comment

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

that's a good point, I'm happy to merge that change if you or @remyroy would like to PR it (assuming no objections?)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm good with @InvisibleSymbol suggestion. Let me do another PR.

bors bot pushed a commit that referenced this pull request Sep 10, 2022
## Proposed Changes

Use a generic domain for community checkpoint sync example to meet the concern raised in #3558 (comment)
bors bot pushed a commit that referenced this pull request Sep 10, 2022
## Proposed Changes

Use a generic domain for community checkpoint sync example to meet the concern raised in #3558 (comment)
Woodpile37 pushed a commit to Woodpile37/lighthouse that referenced this pull request Jan 6, 2024
## Proposed Changes

Add a section on the new community checkpoint sync endpoints in the book. This should help stakers sync faster even without having to create an account.
Woodpile37 pushed a commit to Woodpile37/lighthouse that referenced this pull request Jan 6, 2024
## Proposed Changes

Use a generic domain for community checkpoint sync example to meet the concern raised in sigp#3558 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation ready-for-merge This PR is ready to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants