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 cookie auth if requesting the manifest fails with token auth #641

Merged
merged 2 commits into from
Nov 28, 2022

Conversation

StollD
Copy link
Contributor

@StollD StollD commented Aug 4, 2022

Some content, like Amazon Freevee, is not available when using token auth. By allowing to use cookie auth if token auth does not work, it will be possible to play that content without degrading the quality of anything that is available with token auth.

Admittedly, just stuffing the code into a loop that iterates over [True, False] is a bit of a hack, but it seemed cleaner that just putting the code there a second time. I would have prefered a method, but given that the return values from _getPlaybackVars are used later, this could have become a bit messy. In a loop they will just "leak" to the outside, because of python.

Some content like Amazon Freevee is not available when using token auth.
By allowing to use cookie auth if token auth does not work, it will be
possible to play that content without degrading the quality of anything
that is available with token auth.
@Liqianyu
Copy link

Liqianyu commented Aug 9, 2022

This successfully fixed #640

https://github.com/Sandmann79/xbmc/blob/master/plugin.video.amazon-test/resources/lib/network.py#L278
Hardcoding only Android devicetypeid can support HEVC (H.265) is wrong. I'm sure cookie auth can get FreeVee HEVC (H.265) content.

@Ronny-nerd
Copy link

Thanks for the quick Freevee fix for Android...StollD...this one works well for me.

@Varstahl
Copy link
Collaborator

I thought I had already merged this when I released 0.9.6, I didn't even test it ç_ç
/facepalm

@Varstahl Varstahl merged commit 008bebe into Sandmann79:master Nov 28, 2022
@Varstahl
Copy link
Collaborator

Thank you for the contribution :)

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.

None yet

4 participants