Closed
Bug 895479
Opened 12 years ago
Closed 12 years ago
Strange locale-related redirects at URLs like http://www.mozilla.org/products/firefox
Categories
(www.mozilla.org :: General, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: Aleksej, Assigned: pmac)
References
(Blocks 1 open bug, )
Details
(Whiteboard: r=121357,121358)
Attachments
(1 file)
|
115.48 KB,
text/plain
|
Details |
Strange redirects happen from http://www.mozilla.org/products/firefox/ and derived URLs. Visiting with a Russian Firefox Nightly:
http://www.mozilla.org/products/firefox/ →
http://www.mozilla.org/ru/products/da/firefox/
http://www.mozilla.org/ru/products/firefox/ →
http://www.mozilla.org/ru/produ/ru/firefox/fx/
http://www.mozilla.org/ru/products/firefox →
http://www.mozilla.org/ru/produ/ru/firefox/
http://www.mozilla.org/en-US/products/firefox →
http://www.mozilla.org/en-US/pr/en-US/firefox/
http://www.mozilla.org/products/firefox →
http://www.mozilla.org/ru/products/ru/firefox
http://www.mozilla.org/../products/firefox →
http://www.mozilla.org/ru/products/en-GB/firefox
http://www.mozilla.org/../products/firefox/ →
http://www.mozilla.org/ru/products/ja/firefox/
It happens only with http://www.mozilla.org/[locale]/products/firefox[/] :
http://www.mozilla.org/products/ru/firefox/ →
http://www.mozilla.org/ru/products/ru/firefox/
http://www.mozilla.org/TEST/products/firefox/ →
http://www.mozilla.org/ru/TEST/products/firefox/
| Reporter | ||
Updated•12 years ago
|
Blocks: mozorg-redirects
Comment 1•12 years ago
|
||
Cmore and Pmac - Could you please take a look at this? Thx
Flags: needinfo?(pmac)
Flags: needinfo?(chrismore.bugzilla)
Comment 2•12 years ago
|
||
Hmm, this is strange. Line 1192 in .htaccess here says
Redirect permanent /products/firefox/index.html http://www.mozilla.com/firefox/
mozilla.com redirects to mozilla.org/firefox/ and mozilla.com/firefox/ also redirects to mozilla.org/firefox/.
I can't see where this invalid redirect is coming from.
Flags: needinfo?(chrismore.bugzilla)
Comment 3•12 years ago
|
||
This is a valid bug
try this
curl -H "Accept-Language: ru" -ILA "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:17.0) Gecko/17.0 Firefox/25.0" mozilla.org/ru/products/firefox/
HTTP/1.1 301 Moved Permanently
Server: Apache
X-Backend-Server: bedrock1.webapp.phx1.mozilla.com
Vary: Accept-Encoding
Content-Type: text/html; charset=iso-8859-1
Date: Thu, 17 Oct 2013 20:20:58 GMT
Location: http://www.mozilla.org/ru/products/firefox/
Transfer-Encoding: chunked
Connection: Keep-Alive
X-Cache-Info: caching
HTTP/1.1 301 MOVED PERMANENTLY
Server: Apache
X-Backend-Server: bedrock3.webapp.phx1.mozilla.com
Vary: Accept-Encoding
Cache-Control: max-age=600
Content-Type: text/html; charset=utf-8
Date: Thu, 17 Oct 2013 20:20:58 GMT
Location: http://www.mozilla.org/ru/produ/ru/firefox/new/
Expires: Thu, 17 Oct 2013 20:30:58 GMT
Transfer-Encoding: chunked
Connection: Keep-Alive
X-Frame-Options: DENY
X-Cache-Info: caching
HTTP/1.1 404 Not Found
Server: Apache
X-Backend-Server: bedrock1.webapp.phx1.mozilla.com
Vary: Accept-Encoding
Cache-Control: no-store, no-cache, must-revalidate, post-check=0, pre-check=0, private
Content-Type: text/html; charset=UTF-8
Date: Thu, 17 Oct 2013 20:20:58 GMT
Status: 404 Not Found
Pragma: no-cache
Transfer-Encoding: chunked
Connection: Keep-Alive
X-Robots-Tag: noodp
X-Cache-Info: not cacheable; response specified "Cache-Control: no-store"
Comment 4•12 years ago
|
||
This is broken for many locales. I think there is a problem the locale redirects on the bedrock side
Comment 5•12 years ago
|
||
(In reply to raymond [:retornam] (needinfo? me) from comment #4)
> Created attachment 818662 [details]
> log
>
> This is broken for many locales. I think there is a problem the locale
> redirects on the bedrock side
Yeah, I don't see a specific redirect causing this and there are all sorts of weirdness going on with substrings popping into the redirected address. It is almost like a bad regex on the locale detection. It is just weird that I cannot replicate it with URLs other than /products/firefox/
| Assignee | ||
Comment 6•12 years ago
|
||
I think the culprit is this pass-through rule:
http://viewvc.svn.mozilla.org/vc/projects/mozilla.com/trunk/.htaccess?view=markup#l174
It's just passing requests to /products/firefox/ through to /firefox/, which is being picked up by bedrock and since the requesting URL and the actual URL of the view are different bedrock is getting confused. I think this rule and others around it are now superfluous and should be either removed or converted to redirects.
:cmore do you see anything in these rules that should be simply removed? I think most should just be changed from [PT] to [L,R=permanent].
Flags: needinfo?(pmac) → needinfo?(chrismore.bugzilla)
Comment 7•12 years ago
|
||
Let's just change the PT to L,R=permanent so it drops the pass through URLs. I wouldn't pull anything else out for now.
Flags: needinfo?(chrismore.bugzilla)
| Assignee | ||
Comment 8•12 years ago
|
||
I couldn't reproduce this locally or in my remote dev environment. So I just committed r121357 to trunk so we can see if my theory is correct on dev. We should check dev now to see if the problem still exists. If so, something else is interfering with these requests.
Assignee: nobody → pmac
Status: NEW → ASSIGNED
Whiteboard: r=121357
| Assignee | ||
Updated•12 years ago
|
Whiteboard: r=121357 → r=121357,121358
| Assignee | ||
Comment 9•12 years ago
|
||
Looking a lot better:
$ curl -H "Accept-Language: ru" -IL https://www-dev.allizom.org/products/firefox/ | egrep '^(Location|HTTP)'
HTTP/1.1 301 Moved Permanently
Location: https://www-dev.allizom.org/firefox/
HTTP/1.1 301 MOVED PERMANENTLY
Location: https://www-dev.allizom.org/ru/firefox/
HTTP/1.1 301 MOVED PERMANENTLY
Location: https://www-dev.allizom.org/ru/firefox/new/
HTTP/1.1 200 OK
Let me know what you think retornam. If it's good we can roll it out.
Flags: needinfo?(mozbugs.retornam)
Comment 10•12 years ago
|
||
fixed on dev
curl -H "Accept-Language: ru" -ILA "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:17.0) Gecko/17.0 Firefox/25.0" www-dev.allizom.org/ru/products/firefox/
HTTP/1.1 301 Moved Permanently
Content-Type: text/html
Date: Mon, 21 Oct 2013 23:34:45 GMT
Location: https://www-dev.allizom.org/ru/products/firefox/
Connection: Keep-Alive
Content-Length: 0
HTTP/1.1 301 Moved Permanently
Server: Apache
X-Backend-Server: bedrock1.dev.webapp.phx1.mozilla.com
Vary: Accept-Encoding
Cache-Control: max-age=900
Content-Type: text/html; charset=iso-8859-1
Date: Mon, 21 Oct 2013 23:34:45 GMT
Location: https://www-dev.allizom.org/ru/firefox/
Expires: Mon, 21 Oct 2013 23:49:45 GMT
Transfer-Encoding: chunked
Connection: Keep-Alive
X-Robots-Tag: noodp
HTTP/1.1 301 MOVED PERMANENTLY
Server: Apache
X-Backend-Server: bedrock1.dev.webapp.phx1.mozilla.com
Vary: Accept-Encoding
Cache-Control: max-age=600
Content-Type: text/html; charset=utf-8
Date: Mon, 21 Oct 2013 23:34:45 GMT
Location: https://www-dev.allizom.org/ru/firefox/new/
Expires: Mon, 21 Oct 2013 23:44:47 GMT
Transfer-Encoding: chunked
Connection: Keep-Alive
X-Frame-Options: DENY
HTTP/1.1 200 OK
Server: Apache
X-Backend-Server: bedrock1.dev.webapp.phx1.mozilla.com
Vary: Accept-Encoding
Cache-Control: max-age=600
Content-Type: text/html; charset=utf-8
Date: Mon, 21 Oct 2013 23:34:47 GMT
Expires: Mon, 21 Oct 2013 23:44:49 GMT
Transfer-Encoding: chunked
Connection: Keep-Alive
X-Frame-Options: DENY
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
| Assignee | ||
Comment 11•12 years ago
|
||
Merged to prod in r121454.
Comment 12•12 years ago
|
||
fixed on prod
curl -H "Accept-Language: fr" -ILA "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.7; rv:17.0) Gecko/17.0 Firefox/25.0" www.mozilla.org/ru/products/firefox/
HTTP/1.1 301 Moved Permanently
Server: Apache
X-Backend-Server: bedrock3.webapp.phx1.mozilla.com
Vary: Accept-Encoding
Cache-Control: max-age=900
Content-Type: text/html; charset=iso-8859-1
Date: Tue, 22 Oct 2013 19:34:41 GMT
Location: http://www.mozilla.org/ru/firefox/
Expires: Tue, 22 Oct 2013 19:49:41 GMT
Transfer-Encoding: chunked
X-Robots-Tag: noodp
Connection: Keep-Alive
X-Cache-Info: caching
HTTP/1.1 301 MOVED PERMANENTLY
Server: Apache
X-Backend-Server: bedrock1.webapp.phx1.mozilla.com
Vary: Accept-Encoding
Cache-Control: max-age=600
Content-Type: text/html; charset=utf-8
Date: Tue, 22 Oct 2013 19:34:41 GMT
Location: http://www.mozilla.org/ru/firefox/new/
Expires: Tue, 22 Oct 2013 19:44:41 GMT
Transfer-Encoding: chunked
Connection: Keep-Alive
X-Frame-Options: DENY
X-Cache-Info: caching
HTTP/1.1 200 OK
Server: Apache
X-Backend-Server: bedrock2.webapp.phx1.mozilla.com
Vary: Accept-Encoding
Cache-Control: max-age=600
Content-Type: text/html; charset=utf-8
Date: Tue, 22 Oct 2013 19:34:41 GMT
Expires: Tue, 22 Oct 2013 19:44:41 GMT
Transfer-Encoding: chunked
Connection: Keep-Alive
X-Frame-Options: DENY
X-Cache-Info: caching
Status: RESOLVED → VERIFIED
Flags: needinfo?(mozbugs.retornam)
You need to log in
before you can comment on or make changes to this bug.
Description
•