Skip to content

docs: production guide base setup #804

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
merged 5 commits into from
Dec 10, 2020
Merged

Conversation

vasco-santos
Copy link
Member

@vasco-santos vasco-santos commented Nov 19, 2020

These were started in #718, but doing all of them in the same PR will just get things blocked longer. So, this PR is the first of a set of PRs that should land over time. One per guide to ease reviews and prioritize accordingly.

This PR adds the base README for the production guides.

Needs:

@vasco-santos vasco-santos force-pushed the docs/production-guides-setup branch 2 times, most recently from d3dfb1e to 0cc79e9 Compare November 27, 2020 14:10
* [Production](#production)
* [Star servers](#star-servers)
* [Delegate nodes](#delegate-nodes)
* [Circuit Relay](#circuit-relay)
Copy link
Member Author

Choose a reason for hiding this comment

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


It is worth pointing out that with new discovery protocols on the way, as well as support for distributed signaling, the star servers should be deprecated on the long run.

## Delegate nodes
Copy link
Contributor

Choose a reason for hiding this comment

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

These are IPFS only, we should clearly note that. Our only supported delegates atm leverage the IPFS http api.

* [Circuit Relay](#circuit-relay)
* [SSL](#ssl)

## `webrtc-star` servers
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it would be helpful to more clearly state what environments these are useful for, which is to dial a browser. If that's not really in my user flow, I don't need them.

Also listing alternatives would be helpful. In this instance, relays.


This guide aims to guide you from using the public infrastructure into setting up your own.

## Table of Contents
Copy link
Contributor

Choose a reason for hiding this comment

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

I would reframe this guide. Right now it is focused on the services and less on what the user actually needs. I would think about it in terms of flows. Something like:

  • Joining the Network
    • How do I join the public network, how can I run my own bootstrap nodes? JS isn't currently great for being bootstrap nodes, because we need to improve the DHT first to be an effective server.
  • Connecting to browser nodes
    • This is where webrtc-star and relays come in
  • Querying the network from the browser
    • Delegates go here. In theory the DHT can work, it's just slow and requires WSS or webrtc-star support on all dialable nodes, we just really don't recommend it and if you're using the public dht it's really not going to work.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, I think this makes sense

Copy link
Member Author

Choose a reason for hiding this comment

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

I added a "Others" section where I included the SSL subsection. SSL and other things will be transversal to the flows described

@vasco-santos vasco-santos force-pushed the docs/production-guides-setup branch from dcf0e52 to 5660d8f Compare December 4, 2020 14:50
@vasco-santos vasco-santos marked this pull request as ready for review December 10, 2020 13:39
@vasco-santos vasco-santos merged commit d89b2a0 into 0.30.x Dec 10, 2020
@vasco-santos vasco-santos deleted the docs/production-guides-setup branch December 10, 2020 13:54
vasco-santos added a commit that referenced this pull request Dec 10, 2020
vasco-santos added a commit that referenced this pull request Dec 16, 2020
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