If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Link to Search tools is a 404 in [PN]AMO

VERIFIED FIXED

Status

Infrastructure & Operations
WebOps: Other
--
critical
VERIFIED FIXED
7 years ago
4 years ago

People

(Reporter: krupa, Assigned: oremj)

Tracking

Details

(Whiteboard: [z], URL)

(Reporter)

Description

7 years ago
https://preview.addons.mozilla.org/en-US/firefox/search-tools/ is a 404
Is there a page this comes from or are you just making up URLs?
Target Milestone: 5.11.3 → 5.11.4
(Reporter)

Comment 2

7 years ago
STR:
1.AMO homepage->Click on Search tools in categories menu
This was fixed in bug 570983 until we changed the url to be /search-tools.

New rewrite (we can remove the old one):

RewriteRule
^((?:(?:/\w{2,3}(?:-\w{2})?)(?:/(?:firefox|seamonkey|thunderbird|mobile|sunbird))?)?)/search-tools/.+$ 
               $1/search-engines/ [R,L]
Assignee: nobody → server-ops
Component: Public Pages → Server Operations: Web Content Push
Product: addons.mozilla.org → mozilla.org
QA Contact: web-ui → mrz
Target Milestone: 5.11.4 → ---
Version: 4.x → other
This is a 404 on the front page of the live site now too.  Need some help from IT
Severity: normal → critical
(Assignee)

Updated

7 years ago
Assignee: server-ops → jeremy.orem+bugs
(Assignee)

Comment 5

7 years ago
Made this change in preview, next, and prod:

-    RewriteRule ^((?:(?:/\w{2,3}(?:-\w{2})?)(?:/(?:firefox|seamonkey|thunderbird|mobile|sunbird))?)?)/search-engines/.+$ $1/search-engines/ [R,L]
+    RewriteRule ^((?:(?:/\w{2,3}(?:-\w{2})?)(?:/(?:firefox|seamonkey|thunderbird|mobile|sunbird))?)?)/search-tools/.+$ $1/search-engines/ [R,L]
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
(Assignee)

Comment 6

7 years ago
Looks like that didn't fix it /.+ doesn't match /$

I'm leaving it as is right now, because I don't know what the correct behavior should be. Do I rewrite /search-tools/ to /z/search-tools/?
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Per bug 574164, we're only comfortable with having the Remora search tools page out, so https://preview.addons.mozilla.org/en-US/firefox/browse/type:4
Sorry, that should be /search-tools.*

We're having zamboni/search-tools(/:category)? redirect to remora/search-engines.
(Assignee)

Comment 9

7 years ago
> -    RewriteRule ^((?:(?:/\w{2,3}(?:-\w{2})?)(?:/(?:firefox|seamonkey|thunderbird|mobile|sunbird))?)?)/search-tools/.+$ $1/search-engines/ [R,L]
> +    RewriteRule ^((?:(?:/\w{2,3}(?:-\w{2})?)(?:/(?:firefox|seamonkey|thunderbird|mobile|sunbird))?)?)/search-tools.*$ $1/search-engines/ [R,L]
Status: REOPENED → RESOLVED
Last Resolved: 7 years ago7 years ago
Resolution: --- → FIXED
Verified FIXED; tested all supported apps.
Status: RESOLVED → VERIFIED
Component: Server Operations: Web Operations → WebOps: Other
Product: mozilla.org → Infrastructure & Operations
You need to log in before you can comment on or make changes to this bug.