521287 - Redirect getpersonas.com designer urls

Status: VERIFIED FIXED

(Websites Graveyard :: getpersonas.com, defect)

Description

[Suneel]

Through past promotions, we have been pushing clean links such as getpersonas.com/harrypotter.

Since the last push, this link is now a 404. The proper link is http://www.getpersonas.com/en-US/gallery/Designer/harry_potter. 

Can we redirect to the right link? This would also solve the problem for partners that want to advertise a clean link on their sites.

Comment 1

[Zandr Milewski [:zandr]]

Right, we used to have this:

RewriteRule ^/harrypotter /gallery/designer/harry_potter [PT,L]

In fact, we still do, but localization broke it.

I don't know how localization works, so I don't the right way to fix this.

Handing off to rdoherty, assign it back to me with the right conf changes if necessary.

Comment 2

[Ryan Doherty (:rdoherty)]

Due to l10n we can't have a rewrite rule to a non-l10n url. So it should be a redirect from:

/harrypotter to /gallery/Designer/harry_potter

Then our locale detection code can detect the language and redirect to 
/locale/gallery/designer/harry_potter 

If we are going to have more 'clean' urls in the future we should simply allow users to create their own base url. Or I think we can redirect any urls like:

getpersonas.com/foo to getpersonas.com/locale/gallery/designer/foo

Comment 3

[Zandr Milewski [:zandr]]

So there's a RedirectMatch in the apache conf for harrypotter now. 

With the current app, we need to edit the apache conf in puppet and push it to the webheads for each 'designer url'

Obviously, this won't scale. :)

Comment 4

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

Succinct test case, please?  Thanks!

Comment 5

[Ryan Doherty (:rdoherty)]

Yeah, I want to get all urls into a .htaccess file so we just need a svn up for url changes.

Comment 6

[Ryan Doherty (:rdoherty)]

(In reply to comment #4)
> Succinct test case, please?  Thanks!

Load: http://getpersonas.com/harrypotter
Should redirect to http://www.getpersonas.com/en-US/gallery/designer/harry_potter

Comment 7

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

(In reply to comment #6)
> (In reply to comment #4)
> > Succinct test case, please?  Thanks!
> 
> Load: http://getpersonas.com/harrypotter
> Should redirect to
> http://www.getpersonas.com/en-US/gallery/designer/harry_potter

Oh, this is a prod fix :-)

(I was testing http://sm-personas01.mozilla.org/harrypotter, which is a 404.)

Thanks; Verified FIXED.

Comment 8

[Zandr Milewski [:zandr]]

Fixed the stage Redirect too, though the harry_potter designer doesn't exist on stage.