Closed Bug 1925995 Opened 1 year ago Closed 1 year ago

collinsdictionary.com - The video is not playing with ETP Strict/Standard

Categories

(Web Compatibility :: Privacy: Site Reports, defect, P2)

Firefox 131
Desktop
Windows 11

Tracking

(firefox137 fixed)

RESOLVED FIXED
Tracking Status
firefox137 --- fixed

People

(Reporter: railioaie, Assigned: bvandersloot)

References

(Blocks 1 open bug, )

Details

(Keywords: webcompat:site-report, webcompat:tracker-blocking, Whiteboard: [webcompat-source:product])

Attachments

(1 file)

Environment:
Operating system: Windows 11
Firefox version: Firefox 131.0.3 (release)

Preconditions:

  • ETP set to STRICT/Standard
  • Clean profile

Steps to reproduce:

  1. Navigate to: https://www.collinsdictionary.com/dictionary/english/valuable
  2. Play the video

Expected Behavior:
The video is playing correctly

Actual Behavior:
The video player is not displayed.

Notes:

  • Not reproducible with ETP turned OFF (both Normal and Private Browsing)
  • Reproducible on the latest Nightly

Created from webcompat-user-report:07a62698-63f9-4523-be2e-2cb4ea4376aa

Severity: -- → S2
No longer depends on: tp-breakage
Priority: P3 → P2
Assignee: nobody → tihuang
Status: NEW → ASSIGNED

This is happening because we classify https://openfpcdn.io/botd/v1 (FingerprintJS) as an invasive fingerprinter (rightly), and they are gating loading youtube videos based on it working.

// HCD-5309 - We use BotD library to make sure user is not a bot before playing youtube videos
const botdPromise = import('https://openfpcdn.io/botd/v1').then((Botd) => Botd.load());

window.addEventListener('DOMContentLoaded', function() {
    handleBotDetection();
});

function handleBotDetection() {
    botdPromise
        .then((botd) => botd.detect())
        .then((result) => {
            if (!result.bot) {
                loadYoutubeApi();
                setupYouTubePlayers();
            }
        })
        .catch((error) => {
            console.error("handleBotDetection() " + error);
            document.querySelectorAll('.error-youtube').forEach(function(element) {
                element.style.display = 'flex';
            });

            document.querySelectorAll('.youtube-video').forEach(function(element) {
                element.style.display = 'none';
            });
        });
}

We can try outreach here, but I am tempted to figure out what to shim here

Assignee: tihuang → bvandersloot
No longer blocks: tp-breakage
Pushed by twisniewski@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0a665a4ca633 Shim FingerprintJS Bot Detection - r=twisniewski,webcompat-reviewers
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED

:bvandersloot, I wonder if this should be considered for a beta uplift request given the S2 severity?
Of course that depends on the risk of uplifting the patch vs riding the train with Fx137.
What do you think?

Flags: needinfo?(bvandersloot)

I don't think there is significant risk to uplifting it, especially this early into Beta, but it was unfixed a while before this.

Thoughts Tom on uplifting a shim?

Flags: needinfo?(bvandersloot) → needinfo?(twisniewski)

It's fine to uplift it if we'd like, as the risk should be very low as usual for such uplifts. I'm just not sure if it's really worth the effort (I'll leave it up to others to decide)?

Flags: needinfo?(twisniewski)

No worries if not, it's not a new issue as mentioned but I thought it was worth asking

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: