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

Add our instance please #81

Closed
tio-trom opened this issue Jun 8, 2021 · 24 comments · Fixed by #83
Closed

Add our instance please #81

tio-trom opened this issue Jun 8, 2021 · 24 comments · Fixed by #83

Comments

@tio-trom
Copy link
Contributor

tio-trom commented Jun 8, 2021

This is our instance https://ytb.trom.tf/ - we update it very often and use anti-captcha.com. We test it daily with tens of videos.

@syeopite
Copy link
Member

syeopite commented Jun 8, 2021

What country is the instance hosted in? Also, please enable statistics! The rewrite of the instance list (See #74 or #76) both requires /api/v1/stats to be turned on.

@tio-trom
Copy link
Contributor Author

tio-trom commented Jun 9, 2021

What country is the instance hosted in?

Germany.

Also, please enable statistics!

I did so now.

@unixfox
Copy link
Member

unixfox commented Jun 9, 2021

Your Invidious instance has modified files from the current source code (the home screen is different), you should publish these.

@tio-trom
Copy link
Contributor Author

tio-trom commented Jun 9, 2021

The only modified things are:

  1. Removing the footer with the cryptocurrencies via footer {display:none;} .
  2. And the config file where we may have enabled or disabled features. Should that be also published?

@unixfox
Copy link
Member

unixfox commented Jun 9, 2021

I'm sorry, but the home page doesn't look like the original Invidious home page.

@TheFrenchGhosty Could you explain to @TROMsite the AGPL license and the consequences of this license?

@tio-trom
Copy link
Contributor Author

tio-trom commented Jun 9, 2021

This is a Yunohost default installation. The only thing I did was to remove the cryptocurrencies footer. That's all. How should a homepage look like?

@unixfox
Copy link
Member

unixfox commented Jun 9, 2021

It looks like any empty Invidious instance like this one: https://invidious.namazso.eu

@tio-trom
Copy link
Contributor Author

tio-trom commented Jun 9, 2021

@unixfox
Copy link
Member

unixfox commented Jun 9, 2021

@TheFrenchGhosty knows better about the AGPL license than me, but IIRC ANY changes that differ from the original source code has to be disclosed. Even if it looks almost similar to the original source code.

@tio-trom
Copy link
Contributor Author

tio-trom commented Jun 9, 2021

Ok. It should be noted that I am using a default Yunohost install so maybe it should be reported to Yunohost then.

@unixfox
Copy link
Member

unixfox commented Jun 9, 2021

Ok. It should be noted that I am using a default Yunohost install so maybe it should be reported to Yunohost then.

Don't use Yunohost then, we have a very good documentation that explain how to run Invidious on Docker: https://docs.invidious.io/Installation.md#docker
I'm sorry, but if you aren't capable of using Docker, then you aren't capable of maintaining a public Invidious instance. It takes some Linux administrator knowledge to maintain one, a public Invidious instance receive way more traffic than a private Invidious instance.

We are very happy to welcome new public Invidious instances, but we don't want to have public instances that become not updated after a while and broken. YouTube frequently update their systems, and we often update the Invidious source code to reflect that. So you have to consider that Invidious is not a software that you install and done you don't need to do anything else for a year.

@tio-trom
Copy link
Contributor Author

tio-trom commented Jun 9, 2021

To the contrary, we use this invidious instance daily for our projects and we update it several times a week, pulling via yunohost that works with this git version of invidious. Is up to you to include it in the official list or not. I thought to do some good and suggest our months old instance since we use it daily and are willing to even pay for anti-captcha.com to make sure it works. We test it daily with perhaps hundreds of videos. So far it worked.

Yunohost is an amazing project that allows us to manage and maintain a lot of such free services.

@hackerncoder
Copy link
Contributor

hackerncoder commented Jun 9, 2021

@unixfox, the search page as front page thingy is something we added iv-org/invidious#1977

For AGPL: if you modify the Program, your modified version must prominently offer all users interacting with it remotely through a computer network (if your version supports such interaction) an opportunity to receive the Corresponding Source of your version by providing access to the Corresponding Source from a network server at no charge, through some standard or customary means of facilitating copying of software. (Section 13)

So yes, you need to publish your changes @TROMsite or not change it.

@tio-trom
Copy link
Contributor Author

tio-trom commented Jun 9, 2021

Ok so the frontpage is the same then. In regards to changing anything, I only added the display:none for the footer in a CSS file. It isa visual tweak. You mean even these need to be published? Seems kinda extreme to me, CSS is merely a way to tweak the design but not changing the source code of the software itself - like no functionality change.

@syeopite
Copy link
Member

syeopite commented Jun 9, 2021

Yes. All changes needs to be published with a clear list of what's changed. See the ytprivate repo if you'd like an example.

@TheFrenchGhosty
Copy link
Member

TheFrenchGhosty commented Jun 9, 2021

Exactly, any changes, even minor needs to be properly explained, and released under the same license. We don't make any exception for AGPL violation, yes, even for one line.

CSS is merely a way to tweak the design but not changing the source code of the software itself - like no functionality change.

By changing the CSS, you're changing the software.

@tio-trom
Copy link
Contributor Author

tio-trom commented Jun 9, 2021

Never seen such extreme enforcement of these licenses before. I am very surprised. But ok, up to you. We have many instances of peertube, bibliogram, friendica and a lot more, and they even offer an easy way to add custom CSS to customize your instance as you'd like. Didn't expect the invidious team to be so extreme for just removing the footer with CSS. In any case I removed that line. So is our instance ok now for the official list?

@TheFrenchGhosty
Copy link
Member

Never seen such extreme enforcement of these licenses before. I am very surprised. But ok, up to you.

Because it's not the first time: https://github.com/iv-org/documentation/issues?q=is%3Aissue+label%3A%22agpl+violation

@SamantazFox
Copy link
Member

@TROMsite yes, now it's perfect :)

@tio-trom
Copy link
Contributor Author

Ok that's good so then I'll keep it this way. I removed the footer because I wasn't sure what's with all of the crypto addresses I was worried about that.

@SamantazFox
Copy link
Member

The crypto addresses are the different wallets for the invidious project. We plan to remove that one day and host the donate page at invidious.io.

@tio-trom
Copy link
Contributor Author

Yes I was thinking that's their role, I was just very confused. I thought they may be misleading/inaccurate. I highly suggest that you add a donate button for invidious instead of all of those crypto addresses.

@SamantazFox
Copy link
Member

Yep, this is planned: iv-org/invidious#2032

@unixfox
Copy link
Member

unixfox commented Jan 6, 2023

@tio-trom

Please double-check that you have configured hmac_key in the config file or INVIDIOUS_CONFIG then restart the instance.

You must set a random generated value for the parameter hmac_key:! On Linux, you can generate it using the command pwgen 20 1.

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 a pull request may close this issue.

6 participants