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

Compatibility with Brightcove player #3031

Closed
srirambv opened this issue Jan 21, 2019 · 4 comments
Closed

Compatibility with Brightcove player #3031

srirambv opened this issue Jan 21, 2019 · 4 comments
Assignees
Labels
closed/works-for-me priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes release-notes/include webcompat/not-shields-related Sites are breaking because of something other than Shields. webcompat/shields Shields is breaking a website.

Comments

@srirambv
Copy link
Contributor

Description

Brave has compatibility issues with brightcove player which is used on sites. Video fails to play even when shields is down. This has been an issue on muon as well for good number of sites.

Steps to Reproduce

  1. Visit this link
  2. Try playing video with shields up, play button doesn't work
  3. Disable shields, if you are in luck video player starts but if you aren't goes into an infinite loop

Actual result:

screenshot from 2019-01-21 15-09-58
With shields down when it plays
screenshot from 2019-01-21 15-10-46

Expected result:

Should play irrespective of shields settings. Should have a site hack for brightcove player to make it work

Reproduces how often:

Easy

Brave version (brave://version info)

Brave 0.58.21 Chromium: 71.0.3578.98 (Official Build) (64-bit)
Revision 15234034d19b85dcd9a03b164ae89d04145d8368-refs/branch-heads/3578@{#897}
OS All

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds?
    Yes same on dev/beta

Website problems only:

  • Does the issue resolve itself when disabling Brave Shields?
    Sometimes not always not sure why
  • Is the issue reproducible on the latest version of Chrome?
    Works on Chrome with uBlock Origin. Takes around 8secs before the video starts playing
    screenshot from 2019-01-21 15-16-13

Additional Information

cc: @bbondy

@srirambv srirambv added webcompat/not-shields-related Sites are breaking because of something other than Shields. QA/Yes release-notes/include webcompat/shields Shields is breaking a website. labels Jan 21, 2019
@srirambv srirambv added this to the 1.x Backlog milestone Jan 21, 2019
@mrzealot
Copy link

Spent some time on this as a case study, here's what I found:

The "sometimes it starts, sometimes it doesn't" seems related to the default autoplay blocking (which does pop up in the address bar). Allowing it caused reliable video starts for me, while the default behavior seemed to loop until a initiated a seek event, played pause/play again, etc. These probably overrode the need for autostart by issuing an explicit play command on the video.

The "whether it starts at all" part is 3rd-party cookie related within the shields. Brightcove itself shouldn't require cookies at all (policy), but there is an integration with Conviva that wants to access the local storage (which I assume is considered cookie-like, and is blocked for within-an-iframe scripts by default). This causes an unexpected error in initialization so the player doesn't even get to the loading of the video file (which, again, wouldn't require cookies in itself).

Don't have a clear idea on what we should do with this yet. Maybe a "trusted embedders" list could improve the default 3rd-party blocking behavior, but then the problem becomes the compilation of that list. Detection of these "video not starting" cases could use a crawler, too.

@rebron rebron removed this from the 1.x Backlog milestone Feb 7, 2019
@rebron rebron added the priority/P3 The next thing for us to work on. It'll ride the trains. label Feb 15, 2019
@ryanbr ryanbr reopened this Jan 22, 2020
@ryanbr
Copy link

ryanbr commented Jan 22, 2020

Re-opened. This fix will allow the video player on https://www.aljazeera.com/programmes/countingthecost/2019/01/economic-toll-britain-brexit-chaos-190119082641310.html

@ryanbr
Copy link

ryanbr commented Jun 11, 2020

@srirambv is this resolved? just tested in Release

@ryanbr ryanbr self-assigned this Jul 10, 2020
@ryanbr
Copy link

ryanbr commented Jul 10, 2020

Before closing, did re-test this. Video working, no playback issues.

@ryanbr ryanbr closed this as completed Jul 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed/works-for-me priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes release-notes/include webcompat/not-shields-related Sites are breaking because of something other than Shields. webcompat/shields Shields is breaking a website.
Projects
None yet
Development

No branches or pull requests

4 participants