Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1551)

Issue 47860043: redirect old /web/ to homepage - issue 1272 (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 4 months ago by fedelogy
Modified:
8 years ago
Reviewers:
Graham Percival
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

redirect old /web/ to homepage - issue 1272 Also disable indexing of /web/

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -5 lines) Patch
M Documentation/web/server/lilypond.org.htaccess View 5 chunks +8 lines, -5 lines 2 comments Download
M Documentation/web/server/robots.txt View 1 chunk +2 lines, -0 lines 1 comment Download

Messages

Total messages: 2
Graham Percival
probably ok, but I'm not an expert on .htaccess. Note that Phil will need to ...
10 years, 3 months ago (2014-01-09 09:42:54 UTC) #1
fedelogy
10 years, 3 months ago (2014-01-09 20:52:54 UTC) #2
https://codereview.appspot.com/47860043/diff/1/Documentation/web/server/lilyp...
File Documentation/web/server/lilypond.org.htaccess (right):

https://codereview.appspot.com/47860043/diff/1/Documentation/web/server/lilyp...
Documentation/web/server/lilypond.org.htaccess:75: RewriteRule ^(/*)$
%{ENV:WEB}/ [QSA,L]
On 2014/01/09 09:42:54, Graham Percival wrote:
> I'm not too clear on this stuff.  Did we define ENV:WEB somewhere?

at the beginning of the rewrite rules I see:
SetEnvIf REQUEST_URI .* WEB=/website

WEB used to be /web, then it changed to /website (and someone forgot to edit the
comments accordingly).
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b