launchpad-reviewers team mailing list archive

[Guruprasad <mp+465505@xxxxxxxxxxxxxxxxxx>]

Guruprasad has proposed merging ~lgp171188/launchpad:charm-launchpad-codehosting-fix-double-slash-rewrite-url into launchpad:master.

Commit message:
charm/launchpad-codehosting: Fix a double slash in an apache2 rewrite rule

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~lgp171188/launchpad/+git/launchpad/+merge/465505
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~lgp171188/launchpad:charm-launchpad-codehosting-fix-double-slash-rewrite-url into launchpad:master.

diff --git a/charm/launchpad-codehosting/templates/vhosts/bazaar_https.conf.j2 b/charm/launchpad-codehosting/templates/vhosts/bazaar_https.conf.j2
index 2740a53..deb320d 100644
--- a/charm/launchpad-codehosting/templates/vhosts/bazaar_https.conf.j2
+++ b/charm/launchpad-codehosting/templates/vhosts/bazaar_https.conf.j2
@@ -33,6 +33,6 @@
   RewriteRule ^/robots.txt$ - [L]
   RewriteRule ^/favicon.ico$ {{ loggerhead_static_dir }}/images/favicon.ico [L]
   RewriteRule ^/static/(.*)$ /$1 [L]
-  RewriteRule ^/(.*)$ {{ internal_codebrowse_root }}/$1 [P,L]
+  RewriteRule ^/(.*)$ {{ internal_codebrowse_root }}$1 [P,L]
 </VirtualHost>