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
diff --git a/config/cdn.apache2.conf b/config/cdn.apache2.conf
index d66ee64..3c9f643 100644
--- a/config/cdn.apache2.conf
+++ b/config/cdn.apache2.conf
@@ -1,5 +1,5 @@
1# -*-mode: apache;-*-1# -*-mode: apache;-*-
2<VirtualHost __VHOSTADDRESS__>2<VirtualHost *:80>
3 ServerName __CDNSERVERNAME__3 ServerName __CDNSERVERNAME__
4 __CDNSERVERALIAS__4 __CDNSERVERALIAS__
5 ServerAdmin __WEBMASTEREMAIL__5 ServerAdmin __WEBMASTEREMAIL__
diff --git a/config/seccdn.apache2.conf b/config/seccdn.apache2.conf
index 25c04e8..bf54b73 100644
--- a/config/seccdn.apache2.conf
+++ b/config/seccdn.apache2.conf
@@ -1,5 +1,5 @@
1# -*-mode: apache;-*-1# -*-mode: apache;-*-
2<VirtualHost __VHOSTADDRESS__>2<VirtualHost *:443>
3 ServerName __SECCDNSERVERNAME__3 ServerName __SECCDNSERVERNAME__
4 __SECCDNSERVERALIAS__4 __SECCDNSERVERALIAS__
5 ServerAdmin __WEBMASTEREMAIL__5 ServerAdmin __WEBMASTEREMAIL__

Subscribers

People subscribed via source and target branches