Navigation Menu

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

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

Closed
ader1990 opened this issue Feb 1, 2021 · 7 comments
Labels
FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Milestone

Comments

@ader1990
Copy link

ader1990 commented Feb 1, 2021

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.

@gopherbot gopherbot added this to the Unreleased milestone Feb 1, 2021
@ALTree ALTree added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Feb 1, 2021
@ALTree
Copy link
Member

ALTree commented Feb 1, 2021

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
Copy link
Author

ader1990 commented Feb 1, 2021

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
Copy link
Member

ALTree commented Feb 1, 2021

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
Copy link
Author

ader1990 commented Feb 1, 2021

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
Copy link
Author

ader1990 commented Feb 1, 2021

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

@ALTree
Copy link
Member

ALTree commented Feb 1, 2021

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).

@seankhliao seankhliao added NeedsFix The path to resolution is known, but the work has not been done. and removed NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. labels Feb 2, 2021
@gopherbot
Copy link

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

@golang golang locked and limited conversation to collaborators Feb 2, 2022
passionSeven added a commit to passionSeven/website that referenced this issue Oct 18, 2022
Fix the link for the placeholder video like
CL 203825 did for the runtime selected one.

Fixes golang/go#44037

Change-Id: Ic8b397c2c69f94d8a35e800515825380d06eaa4c
Reviewed-on: https://go-review.googlesource.com/c/website/+/288692
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Alexander Rakoczy <alex@golang.org>
Trust: Alexander Rakoczy <alex@golang.org>
Trust: Carlos Amedee <carlos@golang.org>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Projects
None yet
Development

No branches or pull requests

4 participants