Closed
Bug 1199812
Opened 9 years ago
Closed 9 years ago
Treeherder URLs with /ui/ in them now 404
Categories
(Tree Management :: Treeherder, defect)
Tree Management
Treeherder
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: Gijs, Unassigned)
References
Details
https://treeherder.mozilla.org/ui/#/jobs?repo=mozilla-inbound
gets me "Page not found" instead of a redirect to the new URIs. :-(
Comment 1•9 years ago
|
||
Ah yes it's expected that URL containing the deprecated '/ui/' path no longer work. It's not expected that people are still using them however - it's been 7-10 months and I thought Firefox would rewrite awesomebar entries if a 301 was received? (Just not bookmarks). Even if Firefox doesn't rewrite the history URLs, the awesomebar frequency algorithm should be tracking the post-redirect URL not the original. If it did that, the old URL would disappear off the bottom of suggestions in lieu of the new post-redirect URL, since the latter would get more points surely? This sounds like an awesomebar bug to me? (Unless your entries are actually bookmarks?)
Rough timeline:
* The /ui/ part of the URL was removed from new URLs (ie when visiting site root) 10 months ago (bug 1063411 / bug 1088577)
* trychooser updated to print the new URL 9 months ago (bug 1043880) + wiki pages/other tools etc
* a Treeherder 301 redirect for old URLs added 7 months ago (bug 1124670)
* The 301 redirect was removed this week as part of bug 1151803 (Apache is no longer serving the assets directly)
So for the last 7-10 months all links in bugs and IRC (eg pasted by sheriffs or from trychooser emails) have been of the new form. (And Treeherder expires pushlog/job data after 4 months, so old try run links in bugs won't work regardless of URL.)
Given:
* the length of time here
* that this only makes a difference to the generic per-tree page URLs (ie not &revision= URLs, or site root)
* that Treeherder users are technical
* we're going to get rid or /ui/ URL support when we move to Heroku anyway (given we'd have to use Django redirects instead, due to having to stop using Apache)
...I'm going to wontfix this.
Blocks: 1151803
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
Summary: Treeherder broke my old awesomebar entries → Treeherder URLs with /ui/ in them now 404
Comment 2•9 years ago
|
||
(In reply to Ed Morley (UK public holiday 31st Aug) [:emorley] from comment #1)
> * The 301 redirect was removed this week as part of bug 1151803
Sorry bug 1197799 even.
Updated•9 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•