Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 9119001: infrastructure: Changes for the new website (Closed)

Created:
Jan. 2, 2013, 8:33 a.m. by Felix Dahlke
Modified:
Nov. 8, 2013, 8:04 a.m.
Reviewers:
Wladimir Palant
Visibility:
Public.

Description

These are the NGINX configuration changes required for the new website. As discussed, it works like this: 1. If the URL is / or /lang, redirect to /lang/browser based on the user agent 2. If the URL is /lang/browser, rewrite to /lang and set the browser GET parameter

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+29 lines, -42 lines) Patch
M modules/adblockplusorg/files/adblockplus.org View 3 chunks +29 lines, -42 lines 0 comments Download

Messages

Total messages: 2
Felix Dahlke
Jan. 2, 2013, 8:41 a.m. (2013-01-02 08:41:30 UTC) #1
Felix Dahlke
Jan. 2, 2013, 9:42 a.m. (2013-01-02 09:42:24 UTC) #2
Seems like this was already reviewed: http://codereview.adblockplus.org/9048063/

So I'm closing this one.

Powered by Google App Engine
This is Rietveld