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

Rewrite the public instances list #74

Draft
wants to merge 12 commits into
base: master
Choose a base branch
from
Draft

Rewrite the public instances list #74

wants to merge 12 commits into from

Conversation

TheFrenchGhosty
Copy link
Member

@TheFrenchGhosty TheFrenchGhosty commented May 29, 2021

Rendered version: https://github.com/TheFrenchGhosty/documentation/blob/instances-list-rewrite/Public-Instances.md

This PR is a rewrite of the public instances list from scratch.

  • Instances list is now in a table, making things a LOT easier to read
  • The plain text version of the table is readable (disable word wrap)
  • A lot of information have been added to ease the management (mainly internally) but also so that users are aware of who runs X instance
  • Prerequisites have been rewritten better, and new one have been added

It is "incomplete" in the sense that it's missing the link to some privacy policies, and the owners of the Tor onion, but it's enough for https://github.com/iv-org/instances-api to be adapted to work with it.

It mustn't be merged before https://github.com/iv-org/instances-api is ready to support it. (@syeopite seems to plan to rewrite its parser)


Took me around to do ~2 hours, it was a lot more time consuming that I expected.


Note:

The wording of some categories have to be discussed (mainly "DDos Protection / MITM").

@syeopite
Copy link
Member

https://invidious.048596.xyz is ran by @tenpura-shrimp See #6

@TheFrenchGhosty
Copy link
Member Author

@syeopite You're right, I got confused by this commit: a6fdd0e / #39

Public-Instances.md Outdated Show resolved Hide resolved
Public-Instances.md Outdated Show resolved Hide resolved
@FireMasterK
Copy link
Contributor

Should something like "Instances using a modified source code, are required to publish their source code before they can be added to the list, and must timely publish any later modification in a timely manner." be added to prevent another #72 from happening?

@TheFrenchGhosty
Copy link
Member Author

@FireMasterK

Should something like "Instances using a modified source code, are required to publish their source code before they can be added to the list, and must timely publish any later modification in a timely manner." be added to prevent another #72 from happening?

Done, thanks :)

syeopite added a commit to syeopite/documentation that referenced this pull request May 30, 2021
@syeopite
Copy link
Member

I think we should add an uptime requirement. Any instance that is down most of the time should not be added to the official list imo.

@TheFrenchGhosty
Copy link
Member Author

@syeopite

I think we should add an uptime requirement. Any instance that is down most of the time should not be added to the official list imo.

This is a good idea, should how much should we require? 75%?

@unixfox
Copy link
Member

unixfox commented Jun 2, 2021

@syeopite

I think we should add an uptime requirement. Any instance that is down most of the time should not be added to the official list imo.

This is a good idea, should how much should we require? 75%?

At the very least 90% because it's still like almost 3 days per month of downtime:
image
Source: https://uptime.is/

@TheFrenchGhosty
Copy link
Member Author

TheFrenchGhosty commented Jun 2, 2021

@unixfox

@syeopite

I think we should add an uptime requirement. Any instance that is down most of the time should not be added to the official list imo.

This is a good idea, should how much should we require? 75%?

At the very least 90% because it's still like almost 3 days per months of downtime:
image
Source: https://uptime.is/

Yeah, you're right. 90% is the way to go I guess.

@syeopite
Copy link
Member

syeopite commented Jun 5, 2021

This PR can get merged soon. I've finished the parser rewrite for the instances-api a few days ago. See iv-org/instances-api#30

@syeopite
Copy link
Member

syeopite commented Jun 8, 2021

Owner for https://notyoutube.org is wrong. The actual owner is dejalavidavolar. See #58. But due to #82 it should be removed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Work In Progress
Development

Successfully merging this pull request may close these issues.

4 participants