404 of FlaggedRevs-REL1_41-447bfbc.tar.gz
Closed, ResolvedPublicBUG REPORT
Actions

Assigned To

Authored By

	Lens0021
	May 21 2024, 2:31 AM

Description

Steps to replicate the issue (include links if applicable):

What happens?:

Redirected to a nginx error page which says "404 Not Found"

What should have happened instead?:

The download should be started

Software version (on Special:Version page; skip for WMF-hosted wikis like Wikipedia):

Other information (browser name/version, screenshots, etc.):

	Subject	Repo	Branch	Lines +/-
	Ensure file name uses 7 character rev even if ambigious	labs/tools/extdist	master	+2 -1

Mentioned Here: T340882: Hash collision on ExtDist, occurs 404 Not Found on Extension Distributor

Huh

Looks like we determine file name by doing git rev-parse --short=7 HEAD if 7 characters is not unique it does the shortest unique string.

On the mediawiki side we just always use exactly 7 characters.

So there is a small possibility they might not match

Change #1034212 had a related patch set uploaded (by Brian Wolff; author: Brian Wolff):

[labs/tools/extdist@master] Ensure file name uses 7 character rev even if ambigious

Oh, maybe this is a dupe of T340882 and there is an alternate patch there

Change #1034212 merged by jenkins-bot:

[labs/tools/extdist@master] Ensure file name uses 7 character rev even if ambigious

Bawolff closed this task as Resolved.Fri, May 31, 11:45 AM

Bawolff claimed this task.