Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Fixed Spotify no longer playing music #6881

Closed
echosa opened this issue Jan 26, 2017 · 20 comments
Closed

Fixed Spotify no longer playing music #6881

echosa opened this issue Jan 26, 2017 · 20 comments

Comments

@echosa
Copy link
Contributor

echosa commented Jan 26, 2017

Original title: Spotify no longer plays music #6881 (comment)

Test Plan

  1. Open a new tab and go to spotify.com
  2. Try using the player.
  3. Make sure the Widevine popup is shown.

Original issue description

  • Did you search for similar issues before submitting this one?

Yes

  • Describe the issue you encountered:

Spotify is no longer working. When I go to the web player, it get an error. See screenshot.

There are JS errors. I'll attach a second screenshot showing them.

  • Platform (Win7, 8, 10? macOS? Linux distro?):

OS X 10.11.6

  • Brave Version (revision SHA):

0.12.15 and 0.13.0

  • Steps to reproduce:

    1. Go to play.spotify.com
    2. Login to Spotify
    3. You'll be redirected to open.spotify.com
    4. You should see the error in the screenshot below and songs will not play. You should also see the JS errors in the console.
  • Actual result:

Songs don't play. Error is displayed.

  • Expected result:

Songs should play without error.

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?

Yes

  • Is this an issue in the currently released version?

Yes

  • Can this issue be consistently reproduced?

Yes

  • Screenshot if needed:

screen shot 2017-01-26 at 5 13 14 pm

screen shot 2017-01-26 at 5 18 07 pm

@echosa
Copy link
Contributor Author

echosa commented Jan 26, 2017

The multiple NO_TRACK_PLAYER errors are there because I clicked "play" multiple times. Each click of "play" results in another error.

@cndouglas
Copy link

Any difference turning off some of the shield settings?

@echosa
Copy link
Contributor Author

echosa commented Jan 27, 2017

No different with shields off. In fact, I'd already turned shields off before reloading that taking those screenshots. I forgot to mention that in the initial issue. Sorry about that.

@cndouglas
Copy link

Please try again with version 0.13.0 now that it's out.

@echosa
Copy link
Contributor Author

echosa commented Jan 27, 2017

Not fixed. Still happening in 0.13.0.

@starkos
Copy link

starkos commented Feb 5, 2017

FWIW, this is happening in Musicality too. The Spotify redesign seems to be looking for something Chrome specific, and getting unhappy when it can't find it. Not sure what it is yet though…

@starkos
Copy link

starkos commented Feb 19, 2017

Okay, I found the source of the error, though I'm not sure how to fix it. This code in web-player.js:

n = {
                initDataType: void 0,
                audioCapabilities: [{
                    contentType: 'audio/mp4; codecs="mp4a.40.2"',
                    robustness: ""
                }],
                distinctiveIdentifier: "optional",
                persistenceState: "optional",
                sessionTypes: ["temporary"]
            };

return t.navigator.requestMediaKeySystemAccess("com.widevine.alpha", [n]).then(function(e) {
                return e.createMediaKeys()
            }).then(function(t) {
                return e._mediaKeys = t,
                e._player.setMediaKeys(t)
            }).then(function() {
                return e._trySetServerCertificate()
            });

…works in Chrome, but throws a DOMException "Unsupported keySystem" in Electron.

Electron does provide a way to use Chrome's WideVine plugin. But when I tried this, the Spotify code continued to throw the same exception, even though running the code in the Devtools Console worked fine. Not sure what's going on there.

@luixxiul
Copy link
Contributor

@starkos Brave has the option to enable Widevine on about:preferences#security. Would you please try with the option enabled?

@starkos
Copy link

starkos commented Feb 21, 2017

I don't have access to the redesign myself, I was given access to someone's machine via a screen share to try and figure it out. Not sure when I'll get the chance to try again, but I'm hoping to send them a WideVine-enable build (via the Electron command-line options) shortly.

@echosa, can you give it a try?

@echosa
Copy link
Contributor Author

echosa commented Feb 21, 2017

Spotify works with Widevine enabled! Even with shields up! <3

@luixxiul
Copy link
Contributor

@diracdeltas Is it possible to display Widevine control dialog on Spotify automatically as we do on Netflix?

@cndouglas
Copy link

@luixxiul We need to add Spotify here.

@diracdeltas
Copy link
Member

cc @bbondy on adding widevine

@bbondy bbondy added this to the 0.13.6 milestone Feb 22, 2017
@bbondy bbondy self-assigned this Feb 22, 2017
@bbondy
Copy link
Member

bbondy commented Feb 22, 2017

should be straightforward to add for 0.13.6, I'll check.

@bsclifton
Copy link
Member

Moving to 0.13.7

@bsclifton bsclifton modified the milestones: 0.13.7, 0.13.6 Mar 14, 2017
@bsclifton
Copy link
Member

bsclifton commented Apr 4, 2017

Moving to 0.14.3... @bbondy I know you added the Amazon check- please feel free to pull this back in if you have a solution 😄

@bsclifton bsclifton modified the milestones: 0.14.3, 0.14.2 Apr 4, 2017
@cndouglas cndouglas mentioned this issue Apr 7, 2017
4 tasks
@cndouglas
Copy link

PR: #8111 Note: The PR is a possible solution, and I did not verify if it works.

@bbondy bbondy added this to the 0.14.2 milestone Apr 13, 2017
@bbondy bbondy removed this from the 0.14.3 milestone Apr 13, 2017
@bbondy
Copy link
Member

bbondy commented Apr 13, 2017

Moved back to 0.14.2 since there's not much risk here and alex tried the PR and confirmed fixed.

@alexwykoff
Copy link
Contributor

I wasn't prompted for the Widevine player but it worked anyway.

@srirambv
Copy link
Collaborator

Shows the prompt immediately on visit similar to Netflix

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.