Closed
Bug 797577
Opened 12 years ago
Closed 12 years ago
Redirect /contact & /developer
Categories
(www.mozilla.org :: Legacy PHP system, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: bensternthal, Assigned: bensternthal)
References
Details
(Whiteboard: r=109598)
Attachments
(1 file)
1.27 KB,
patch
|
sancus
:
review+
rik
:
review+
|
Details | Diff | Splinter Review |
Putting this in one bug so the patch can be used for both. Redirect mozilla.org/developer to https://developer.mozilla.org/docs/Developer_Guide Redirect mozilla.org/contact/ to /about/contact.html
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → bsternthal
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #667668 -
Flags: review?(sancus)
Comment 2•12 years ago
|
||
Comment on attachment 667668 [details] [diff] [review] Patch to redirect /developer & /contact from legacy mozilla.org Looks fine.
Attachment #667668 -
Flags: review?(sancus) → review+
Assignee | ||
Updated•12 years ago
|
Whiteboard: r=109598
Assignee | ||
Comment 3•12 years ago
|
||
Tested on dev, looks good.
Comment 4•12 years ago
|
||
Comment on attachment 667668 [details] [diff] [review] Patch to redirect /developer & /contact from legacy mozilla.org Review of attachment 667668 [details] [diff] [review]: ----------------------------------------------------------------- Actually, I'll r- this because it doesn't we also need to remove the files in the .org codebase. Also the regex should take care of the case where people access this with index.html in the path. Sorry :(
Attachment #667668 -
Flags: review-
Comment 5•12 years ago
|
||
s/because it doesn't we also/because we also/
Comment 6•12 years ago
|
||
Comment on attachment 667668 [details] [diff] [review] Patch to redirect /developer & /contact from legacy mozilla.org Review of attachment 667668 [details] [diff] [review]: ----------------------------------------------------------------- Nevermind, I'm dumb (© mkelly)
Attachment #667668 -
Flags: review- → review+
Comment 7•12 years ago
|
||
Merged in stage with r109599 and in production with r109600. When this is confirmed to work on production, we can remove the files in the .org codebase.
Comment 8•12 years ago
|
||
it doesn't work with the locale code for example http://www.mozilla.org/en-US/developer should go to https://developer.mozilla.org/docs/Developer_Guide. It is a 404 currently . Same thing for the contact page en-US/contact is a 404 curl -I http://www.mozilla.org/developer HTTP/1.1 301 Moved Permanently Server: Apache X-Backend-Server: bedrock1.webapp.scl3.mozilla.com Vary: Accept-Encoding Cache-Control: max-age=900 Content-Type: text/html; charset=iso-8859-1 Date: Wed, 03 Oct 2012 23:50:12 GMT Location: https://developer.mozilla.org/docs/Developer_Guide Expires: Thu, 04 Oct 2012 00:05:12 GMT Transfer-Encoding: chunked X-Robots-Tag: noodp Connection: Keep-Alive X-Cache-Info: caching
Comment 10•12 years ago
|
||
Rik: Can you push 109602 live when you get a chance? That should resolve this bug.
Assignee | ||
Comment 11•12 years ago
|
||
This looks to be in production. I will resolve this one and file the new ones to delete the old content. Progress!
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 12•12 years ago
|
||
crap did not see the bug about this not working with the language... reopening.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 13•12 years ago
|
||
Raymond: The old .org fondation codebase never supported locales like that so we don't have to cover this case here. Sancus: Can you revert r109602 ? Also, for next redirects, if you need to include the locale, I recommend copying the crazy things in https://svn.mozilla.org/projects/mozilla.com/trunk/.htaccess : ^(\w{2,3}(?:-\w{2}(?:-mac)?)?/)?
Comment 14•12 years ago
|
||
reverted in r109630. That also makes this fixed, since the original bit already went live.
Status: REOPENED → RESOLVED
Closed: 12 years ago → 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•