Closed
Bug 1124670
Opened 10 years ago
Closed 10 years ago
301 redirect treeherder.m.o/ui/* to /*
Categories
(Tree Management :: Treeherder: Infrastructure, defect, P2)
Tree Management
Treeherder: Infrastructure
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: fubar)
References
Details
Bug 1063411 meant that the UI is served out of the domain root, rather than /ui/*
However the old style URLs don't redirect, so people are still pasting them everywhere.
Please could you set up a 301 redirect on stage/prod? :-)
Assignee | ||
Comment 1•10 years ago
|
||
Just to double check, this is what I think is needed:
RewriteRule ^/ui([^/]*) https://treeherder.mozilla.org/$1 [R=301]
I wanted to avoid putting the FQDN in there but you end up with double slashes if you don't. OTOH, maybe there's a different way... tested on one stage node and it appeared to work correctly.
Reporter | ||
Comment 2•10 years ago
|
||
I think this would this work?
RewriteRule ^ui/(.*) $1 [R=301]
(Tested using http://htaccess.madewithlove.be/ )
Assignee | ||
Comment 3•10 years ago
|
||
Hmph, apache docs explicitly say an unqualified/unanchored substitution is invalid; plus, I think I tested it with another flag that made it really not work.
Tried your suggestion and got "The requested URL /ui/ was not found on this server."
But this works:
RewriteRule ^/ui(.*) $1 [R=301]
It's in place on staging and I don't see anything broken. If it's good, let me know and I'll push to prod.
Reporter | ||
Comment 4•10 years ago
|
||
Thank you - looks fine on stage to me :-)
Assignee | ||
Comment 5•10 years ago
|
||
Prod updated in r99799. Puppet will apply within an hour.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•