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

use new v6 URL for rss #2991

Merged
merged 1 commit into from
Oct 10, 2023
Merged

use new v6 URL for rss #2991

merged 1 commit into from
Oct 10, 2023

Conversation

tzarebczan
Copy link
Contributor

We can probably craft these manually again, will try that later (vs calling get)

We can probably craft these manually again, will try that later (vs calling get)
@tzarebczan tzarebczan changed the title use new v6 URL use new v6 URL for rss Oct 10, 2023
@tzarebczan tzarebczan merged commit 03e45ba into master Oct 10, 2023
2 checks passed
@tzarebczan tzarebczan deleted the rss-fix-3 branch October 10, 2023 17:09
@OmarLipton
Copy link

The V6 RSS feeds seem to only work about 10% of the time in VLC and any other media player.
I have old playlists with the V3 URLs and they work perfectly.
The streams with the V6 URLS (https://player.odycdn.com/v6/streams/...) either drop out while playing or don't work at all.
The streams with the V3 URLs (https://player.odycdn.com/api/v3/streams/free/...) work perfectly 100% of the time even when seeking the videos to random positions.
I tried several podcast players, GPodder, Grover, VLC and the RSS based videos either don't play or disconnect.
After the video stops playing, it can't be restarted again.
Why bother with the change to V6?

@tzarebczan
Copy link
Contributor Author

Hmm, we haven't heard other complaints about this yet. They're using the same exact service to provide the data, the only difference is the syntax of the URL. A common v6 URL between how we stream directly on Odysee and via other services like RSS gives us the best caching scenario - otherwise v3 would need to get cached separately by the CDN.

We'll give this a test...

@tzarebczan
Copy link
Contributor Author

We put it back to v3 for now, we had some abuse checks that were messing with v6 + rss.

@tzarebczan
Copy link
Contributor Author

Appreciate the heads up!

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.

2 participants