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

Status: VERIFIED FIXED

(Infrastructure & Operations Graveyard :: WebOps: Other, task)

Description

[krupa raj[:krupa]]

https://preview.addons.mozilla.org/en-US/firefox/search-tools/ is a 404

Comment 1

[Wil Clouser [:clouserw]]

Is there a page this comes from or are you just making up URLs?

Comment 2

[krupa raj[:krupa]]

STR:
1.AMO homepage->Click on Search tools in categories menu

Comment 3

[Jeff Balogh (:jbalogh)]

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]

Comment 4

[Wil Clouser [:clouserw]]

This is a 404 on the front page of the live site now too.  Need some help from IT

Comment 5

[Jeremy Orem [:oremj]]

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]

Comment 6

[Jeremy Orem [:oremj]]

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/?

Comment 7

[Stephen Donner [:stephend] Not actively reading bugmail]

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

Comment 8

[Jeff Balogh (:jbalogh)]

Sorry, that should be /search-tools.*

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

Comment 9

[Jeremy Orem [:oremj]]

> -    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]

Comment 10

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified FIXED; tested all supported apps.