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 all (HTTP(S)) Invidious instances #2488

Merged
merged 7 commits into from
Sep 12, 2019
Merged

Add all (HTTP(S)) Invidious instances #2488

merged 7 commits into from
Sep 12, 2019

Conversation

2secslater
Copy link
Contributor

Public Invidious instances from https://github.com/omarroth/invidious/wiki/invidious-instances have been added to the AndroidManifest.xml to allow opening of the public Invidious (not just invidio.us) links in NewPipe.

Public Invidious instances from https://github.com/omarroth/invidious/wiki/invidious-instances have been added to the AndroidManifest.xml to allow opening of the public Invidious (not just `invidio.us`) links in NewPipe.
@theScrabi
Copy link
Member

Did you add support for this in the extractor? If not you need to otherwise its not working.

@Serkan-devel
Copy link

The APIs do quite differ between that and YT

@theScrabi
Copy link
Member

Thats not the issue however the StreamLinkHandlerFactories in the extractor have to know the invidious instances as well, otherwise the app will accept the links but the backend would not know what to do with them.

@2secslater
Copy link
Contributor Author

I've added the domains to the link factory for YouTube. TeamNewPipe/NewPipeExtractor#182

@Perflyst
Copy link

Perflyst commented Sep 4, 2019

@theScrabi can we get this merged for the next release?

@TobiGr TobiGr merged commit 6a37dfc into TeamNewPipe:dev Sep 12, 2019
This was referenced Sep 12, 2019
@airon90 airon90 mentioned this pull request Sep 14, 2019
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.

5 participants