x/website: https://golang.org featured video is unavailable https://youtu.be/cQ7STILAS0M

[ader1990]

The featured video from https://golang.org (bottom-left of the page) is unavailable. The youtube link is https://www.youtube.com/watch?v=cQ7STILAS0M.

[ALTree]

Thanks for reporting this.

It appears that that video is actually just the same Pike talk of another video (Simplicity is Complicated" talk from dotGo 2015) that is working correctly, so in golang/website@7323790 the link to cQ7STILAS0M was replaced with a link to the latter video.

Looking at the relevant code, it seems the video that is now broken is actually not in the list of the ones that are choosen at random; but the broken video is still used as a placeholder in the html, so if you browse golang.org with javascript disabled, the script that replaces the placeholder video with one of the working videos doesn't run, and you get the static placeholder, which unfortunately now points to a dead video.

I cannot get the broken video by refreshing the page, but if I disable javascript I indeed get that one. Do you have javascript disabled?

In any case the default video that is served when the script doesn't run should be replaced with a link to a working video.

[ader1990]

Hello @ALTree,

I do not have JS disabled, it just appears unavailable. I tried to refresh (CTRL+R) a couple of times, and some featured videos were available, but at some point the unavailable one appeared too.

[ALTree]

Interesting, I don't see how could you be served with the broken link looking at the code (it's not in the list of the videos), but I don't really know javascript so I may be missing something.

cc @dmitshur

[ader1990]

It seems there is some sort of concurrency bug on Chrome only (I can repro it sporadically). I cannot reproduce with Firefox. Google Chrome version used: Version 88.0.4324.104 (Official Build) (64-bit) on Windows 10.

[ader1990]

I could repro the issue on a MacBook Pro with Chrome.

[ALTree]

Oh... so maybe a race between the javascript code substituting the video, and the static value being shown (which points to a video it shouldn't be shown).

[gopherbot]

Change https://golang.org/cl/288692 mentions this issue: content/static/doc: update placeholder video