-
Notifications
You must be signed in to change notification settings - Fork 455
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
Broken: plugin.video.curiositystream - seems API changed #4208
Comments
…less tolerant, reference is: xbmc#4208
@GMaxera Not sure if you've seen this, I'm happy to pick up PRing this into the main repo and stuff if you don't want to do it etc |
No, I didn't see, I'm sorry. If you can pick it up and fix it, please do it. |
No problem, I'll probably pull it into its own repository and stuff and we can sort out getting it all consistent once it's working again |
I've PR'd in a fix for it. The code for just the plugin is here: https://github.com/mintsoft/plugin.video.curiositystream |
This can be closed now; the addon is fixed as of June and I've taken over as maintainer of this addon for the forseeable |
@basrieter Can you close this issue for me? This was fixed in: #4400 |
Add-on details:
Ownership
Broken because:
Seems the Curiosity Stream API may have changed, and I was getting some error messages. I'm too busy to include logs or every detail, but I'll share what I discovered, and how I fixed it. Hopefully this will help someone with the same issue. Or if someone really wants, I can make the changes and open a PR.
Issue:
IndexError
because of empty streams listError originates from last line below (router.py):
Fix: Adapt to seeming API changes (curiositystream.py)
I solved this using postman to check API responses. Two problems. First, the API wasn't happy with the User-Agent provided by the Python requests package. Give it something it does like:
Second, the "type" field of the response showed "HD" which (pretty sure) wont match with
== "hd"
:Issue: Kodi can't play the "master_playlist_url"
Maybe the API changed the provided file from
.m3u8
to.mpd
. Kodi seemed confused.Fix: Use
inputstream.adaptive
(router.py)A proper solution would probably check to make sure it's really an
mpd
file first, but things seem to be working consistently for me.Hope this helps!
The text was updated successfully, but these errors were encountered: