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

The extra.filename property is missing from the callback argument of the defineSubtitlesHandler method. #289

Open
wujekbogdan opened this issue Jul 21, 2024 · 5 comments

Comments

@wujekbogdan
Copy link

According to the defineSubtitlesHandler docs documentation, the callback function's argument should include an extra.filename property. While the extra object is present, it only contains videoHash and videoSize, vut filename is always undefined.

builder.defineSubtitlesHandler((args) => {
  console.log(args.extra.filename); // undefined
  // ...
})
@jaruba
Copy link
Member

jaruba commented Jul 21, 2024

what stremio client app are you using for tests? what platform is it from? what version is the app?

the extra properties may or may not be included depending on the app and its capabilities, wether it is connected to a local streaming server or not, this is why it is called "extra", but we cannot assist or investigate without knowing the exact platform and app version that is making the requests to the addon

@wujekbogdan
Copy link
Author

wujekbogdan commented Jul 21, 2024

  • I'm not using a local streaming server. I'm streaming from Torrentio + Real Debrid
  • macOS desktop app
  • Streamio 4.4.168
  • WebUI 4.4.172
  • Server 4.20.8

// EDIT:
I've just tried a few public domain movies, and the filename property is there.

@dexter21767-dev
Copy link
Member

I've just tried a few public domain movies, and the filename property is there.

@wujekbogdan the desktop app has built-in local streaming server.
the streaming server is responsible for making torrents streamable. and while it's doing that it also gets the filename from the torrent which is passed to the subtitles addon.
but for HTTP streams, the addon that returns the stream is responsible for providing the filename.

Torrentio / public domain movies addons provide torrents.
while Torrentio + RD provides HTTP streams.

@wujekbogdan
Copy link
Author

Does it mean then it is technically impossible to retrieve files names from HTTP streams or is it just outside Streamio control and has to be addressed by the plugin developer?

@wujekbogdan
Copy link
Author

Let me bump the thread. I found a similar issue here: mhdzumair/MediaFusion#214.

From what I understand, it's not that HTTP streams can't provide the file name; it seems that this feature is missing in stremio-core, but it’s technically feasible. Am I right? Should I report the issue on streamio-core?

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

No branches or pull requests

3 participants