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

Let's get rid of Flash [Due: 09/2018] #33

Closed
2 tasks
jiri-janousek opened this issue Mar 11, 2018 · 10 comments
Closed
2 tasks

Let's get rid of Flash [Due: 09/2018] #33

jiri-janousek opened this issue Mar 11, 2018 · 10 comments
Labels
type: enhancement A new feature or an improvement of already present feature.

Comments

@jiri-janousek
Copy link
Member

Since this script is compatible with the Chromium backend, we could proceed further: Replace "requirements": "Codec[MP3] Feature[Flash]" with "requirements": "Codec[MP3] Feature[MSE] Chromium[63]". For this, we need to check:

  • Playback works with Flash plugin disabled (Feature[Flash] removed). I guess this will be okay presuming that Amazon detected functional MSE and didn't bother to use Flash plugin at all during your tests of the Chromium backend.
  • There are no issues under Wayland, such as Port to Chromium Embedded Framework nuvolaplayer#372 (comment).
@jiri-janousek jiri-janousek added the type: enhancement A new feature or an improvement of already present feature. label Mar 11, 2018
@a-m-s
Copy link
Contributor

a-m-s commented Mar 11, 2018

Sorry, flash is required. This is what I get with your suggested requirements:
screenshot from 2018-03-11 23-02-04
Without Flash you can only play music that you own; it won't play music "included with Prime".

@jiri-janousek
Copy link
Member Author

Oh, that's a bummer. I've expected more from Amazon. Could you check whether Flash is also required in the genuine Chrome and then ask Amazon Cloud Player support whether there are any plans to get rid of Flash? You can say that Chrome makes more and more difficult to use Flash and there is a roadmap for its removal.

Meantime, we could switch to Chromium, where Flash is more stable. Could you test following requirements under Wayland? The first line is a comment linking this issue for me not to forget Flash is still required.

"#requirements": "https://github.com/tiliado/nuvola-app-amazon-cloud-player/issues/33",
"requirements": "Codec[MP3] Feature[MSE] Feature[Flash]",

@a-m-s
Copy link
Contributor

a-m-s commented Mar 12, 2018

I get the same message in Chrome. I will send them a message.

@a-m-s
Copy link
Contributor

a-m-s commented Mar 13, 2018

I wrote an email to Amazon support. I got a standard reply ignoring my questions and telling me to clear my cookies or use a different browser.

Trying again, worded for idiots.

@a-m-s
Copy link
Contributor

a-m-s commented Mar 13, 2018

Still no luck. They keep telling me to call them. I don't fancy my chances of getting any operator that has a clue.

@jiri-janousek
Copy link
Member Author

The Amazon support seems to be unbelievably incompetent :-(

@jiri-janousek
Copy link
Member Author

Let's check the progress every 4 months - next check September 2018.

@jiri-janousek jiri-janousek changed the title Let's get rid of Flash Let's get rid of Flash [Due: 09/2018] May 7, 2018
@jiri-janousek jiri-janousek added the stalled Some work has been done but nobody is working on finishing this ticket. label Jun 18, 2018
@jiri-janousek jiri-janousek added status: in progress and removed stalled Some work has been done but nobody is working on finishing this ticket. labels Jan 18, 2019
@a-m-s
Copy link
Contributor

a-m-s commented Jan 25, 2019

I believe this issue is now resolved too! :-D

@jiri-janousek
Copy link
Member Author

Sure :-)

@jiri-janousek
Copy link
Member Author

Released with Nuvola 4.16.0, more info: Nuvola 4.16 Brings Libre.FM Scrobbling, Ivoox Service, and more

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement A new feature or an improvement of already present feature.
Projects
None yet
Development

No branches or pull requests

2 participants