Merge ~kumy/libravatar:bug-1781164 into ~libravatar/libravatar:master

Proposed by Mathieu Alorent
Status: Needs review
Proposed branch: ~kumy/libravatar:bug-1781164
Merge into: ~libravatar/libravatar:master
Diff against target: 22 lines (+2/-2)
2 files modified
config/cdn.apache2.conf (+1/-1)
config/seccdn.apache2.conf (+1/-1)
Reviewer Review Type Date Requested Status
Libravatar Hackers Pending
Review via email: mp+349341@code.launchpad.net
To post a comment you must log in.
~kumy/libravatar:bug-1781164 updated
b28dd61... by Mathieu Alorent <email address hidden>

LP: #1781164 Replace virtualhost placeholder

Revision history for this message
François Marier (fmarier) wrote :

Looking at the config as installed on the production server, I see:

  $ head /etc/apache2/sites-available/libravatar-seccdn.conf
  # -*-mode: apache;-*-
  <VirtualHost *:443>

So this seems to be working fine. Note that the debconf question which replaces the placeholder is set to priority "medium", so you might need to "dpkg-reconfigure -pmedium libravatar-seccdn" to see that question.

Moreover, the default is "*:443":

  Template: libravatar-seccdn/vhostaddress
  Type: string
  Default: *:443
  Description: Apache vhost address for the secure CDN:

So I'm not sure why it's not working for you.

Unmerged commits

b28dd61... by Mathieu Alorent <email address hidden>

LP: #1781164 Replace virtualhost placeholder

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/config/cdn.apache2.conf b/config/cdn.apache2.conf
2index d66ee64..3c9f643 100644
3--- a/config/cdn.apache2.conf
4+++ b/config/cdn.apache2.conf
5@@ -1,5 +1,5 @@
6 # -*-mode: apache;-*-
7-<VirtualHost __VHOSTADDRESS__>
8+<VirtualHost *:80>
9 ServerName __CDNSERVERNAME__
10 __CDNSERVERALIAS__
11 ServerAdmin __WEBMASTEREMAIL__
12diff --git a/config/seccdn.apache2.conf b/config/seccdn.apache2.conf
13index 25c04e8..bf54b73 100644
14--- a/config/seccdn.apache2.conf
15+++ b/config/seccdn.apache2.conf
16@@ -1,5 +1,5 @@
17 # -*-mode: apache;-*-
18-<VirtualHost __VHOSTADDRESS__>
19+<VirtualHost *:443>
20 ServerName __SECCDNSERVERNAME__
21 __SECCDNSERVERALIAS__
22 ServerAdmin __WEBMASTEREMAIL__

Subscribers

People subscribed via source and target branches