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.

Sign up for GitHub

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

Jump to bottom

Add Stremio web support #23

Open
sleeyax opened this issue Aug 2, 2024 · 3 comments
Open

Add Stremio web support #23

sleeyax opened this issue Aug 2, 2024 · 3 comments
Labels
feature New feature

Comments

@sleeyax
Copy link
Owner

sleeyax commented Aug 2, 2024

https://www.reddit.com/r/StremioAddons/s/J63gc8jrld
@sleeyax
Copy link
Owner Author

sleeyax commented Aug 2, 2024

The url returned by the addon is a private url which requires authentication and redirects the player to the actual public video url via the Location header in the http response. If we want to support web we must extract this public url from the response and return the resulting public url instead.

sleeyax added a commit that referenced this issue Aug 2, 2024
@sleeyax
Resolve public stream URLs to skip auth header requirement (see #23)
25da491 
WIP: Although the current solution introduced in this commit works, it can result in a very slow stream list response depending on the number of returned search results.

Alternative approaches:
- Limit the amount of search results
- Add an option to toggle Stremio web support (disabled by default) with the mention that it comes with a speed penalty
- Add an extra endpoint to resolve the stream location on demand (i.e. when the user clicks on it). This is the approach the original Easynews addon takes. I'm currently in favor of this approach but it'll be some extra work to add this endpoint (addon needs conversion to express.js)
@sleeyax sleeyax added the feature New feature label Aug 2, 2024
@sleeyax sleeyax changed the title Support stremio web (?) Add Stremio web support Aug 2, 2024
@sleeyax sleeyax mentioned this issue Sep 30, 2024
Open
@sleeyax
Copy link
Owner Author

sleeyax commented Nov 11, 2024

Important tot note that the inner public stream should be extracted on a per-stream basis. Resolving all of them beforehand (before returning the streams object that is) would take too long.

We likely need to convert from the stremio-addon-sdk to a custom express.js (or similar) setup in order to support this.

@keplx
Copy link

keplx commented Dec 24, 2024

web interface support would be awesome

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature New feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sleeyax @keplx