Bug 249672 - commits.webkit.org cannot handle canonical URL https://commits.webkit.org/253173.313@webkitglib/2.38
Summary: commits.webkit.org cannot handle canonical URL https://commits.webkit.org/253...
Status: REOPENED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Website (show other bugs)
Version: WebKit Nightly Build
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-12-20 13:10 PST by Michael Catanzaro
Modified: 2023-05-25 18:01 PDT (History)
7 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Catanzaro 2022-12-20 13:10:55 PST
The canonical URL for commit https://github.com/WebKit/WebKit/commit/3e9403ce54e3df588a923cc4e6625c8623f9febe (as provided by 'git-webkit canonicalize') is https://commits.webkit.org/253173.313@webkitglib/2.38. But this URL just redirects to https://github.com/WebKit/WebKit/commits instead of actually displaying the commit.

I wonder if the problem is the / character in the branch name?
Comment 1 Radar WebKit Bug Importer 2023-02-06 03:42:03 PST
<rdar://problem/105076804>
Comment 2 Jonathan Bedard 2023-04-20 08:57:09 PDT
This looks to be resolved. Which is a bit weird because I don't believe I changed anything.

Possibly our commits.webkit.org checkouts didn't used to have this branch, but do now? (Which is the other reason commits.webkit.org might not redirect to a specific commit)
Comment 3 Michael Catanzaro 2023-04-20 09:08:55 PDT
Huh, yeah it certainly seems to be fixed. Strange, but nice.

Adrian, this means we can now start running 'git-webkit canonicalize' before pushing commits to the stable branch.
Comment 4 Michael Catanzaro 2023-05-25 14:41:14 PDT
(In reply to Jonathan Bedard from comment #2)
> Possibly our commits.webkit.org checkouts didn't used to have this branch,
> but do now? (Which is the other reason commits.webkit.org might not redirect
> to a specific commit)

Perhaps indeed? Because 2.40 branch links are all broken, e.g.  https://commits.webkit.org/260527.340@webkitglib/2.40

For 2.38, https://commits.webkit.org/253173.447@webkitglib/2.38 works fine but https://commits.webkit.org/253173.448@webkitglib/2.38 does not work. So I think commits.webkit.org is just not fetching all the branches and latest commits from them. It looks like it only knows about 2.38 changes until January.
Comment 5 Alexey Proskuryakov 2023-05-25 15:38:03 PDT
> https://commits.webkit.org/253173.448@webkitglib/2.38 does not work.

Seems to work for me now, FWIW. It redirects to https://github.com/WebKit/WebKit/commit/1b868d373bede773c6d53049a1904ed5b729f44c
Comment 6 Michael Catanzaro 2023-05-25 18:01:56 PDT
Huh, weird.

I just tested and https://commits.webkit.org/253173.452@webkitglib/2.38 worked but https://commits.webkit.org/253173.453@webkitglib/2.38 did not. But now, about one minute later, .453 and .454 are both working and .455 is the first that doesn't work. So the available commit range is somehow expanding.