plugins.vimeo: fix missing HLS/DASH API results#5854
Conversation
|
Thx @bastimeyer ! I sideloaded the modified plugin to streamlink 6.6.2 and tested a live link (selected from here https://vimeo.com/search?live=now) and a stored one and they both worked OK. Thanks a lot for the prompt solution! |
|
@bastimeyer @WolfganP Even with sideloading the newest vimeo.py plugin, I'm still getting the same error. I'm using the latest master version of the file from here: https://github.com/streamlink/streamlink/blob/master/src/streamlink/plugins/vimeo.py Any idea? Thanks a lot for your support. I'm using streamlink 6.7.0, so I assume that the fix was anyway already merged. Without using the manual plugin I end up with the same error. |
|
These changes have made it into the latest stable release, so sideloading is not necessary. |
Fixes #5702
HLS and DASH stream data is set to optional in the API response validation schema:
https://github.com/streamlink/streamlink/blob/6.6.2/src/streamlink/plugins/vimeo.py#L56-L57
If the data is missing from the API response, then its value will be
None, hence theAttributeErrorwhen trying to call.values()for iterating over the data.Didn't check this on actual live event streams, because I don't know how to find them on Vimeo. This PR fixes the
AttributeError, but I don't know if live events are actually working.@WolfganP @Sparticuz
Please see if there are any other issues while trying to watch live event streams, thanks.
https://github.com/streamlink/streamlink/blob/master/CONTRIBUTING.md#pull-request-feedback