Dzahn has submitted this change and it was merged.

Change subject: etherpad - use ssl_ciphersuite
......................................................................


etherpad - use ssl_ciphersuite

In the past, when we changed SSL cipher settings
we had to create many changes, usually one per service.

Then Giuseppe introduced ssl_ciphersuite in I9bc1104b7f770d9
which moves the settings into a centralized function.

This change makes it actually use the new function.

One last time we are going through all of the services,
but in the future we will be able make a single change
to update ciphers across misc. services.

Change-Id: Id35d5235d2ac3141984f3fa6e5991f15c4f00030
---
M manifests/role/etherpad.pp
M templates/misc/etherpad.wikimedia.org.erb
2 files changed, 3 insertions(+), 3 deletions(-)

Approvals:
  jenkins-bot: Verified
  Dzahn: Looks good to me, approved



diff --git a/manifests/role/etherpad.pp b/manifests/role/etherpad.pp
index 47e683c..d62ddd3 100644
--- a/manifests/role/etherpad.pp
+++ b/manifests/role/etherpad.pp
@@ -27,6 +27,8 @@
         }
     }
 
+    $ssl_settings = ssl_ciphersuite('apache-2.2', 'compat')
+
     class { '::etherpad':
         etherpad_host    => $etherpad_host,
         etherpad_ip      => $etherpad_ip,
diff --git a/templates/misc/etherpad.wikimedia.org.erb 
b/templates/misc/etherpad.wikimedia.org.erb
index f0c61ca..a3a22c7 100644
--- a/templates/misc/etherpad.wikimedia.org.erb
+++ b/templates/misc/etherpad.wikimedia.org.erb
@@ -51,12 +51,10 @@
     ServerName <%= @etherpad_host %>
 
     SSLEngine on
-    SSLProtocol +ALL -SSLv2
-    SSLCipherSuite 
ECDHE-RSA-AES128-GCM-SHA256:ECDHE-ECDSA-AES128-GCM-SHA256:ECDHE-RSA-AES256-GCM-SHA384:ECDHE-ECDSA-AES256-GCM-SHA384:ECDHE-RSA-AES128-SHA256:ECDHE-ECDSA-AES128-SHA256:ECDHE-RSA-AES128-SHA:ECDHE-ECDSA-AES128-SHA:ECDHE-RSA-AES256-SHA384:ECDHE-ECDSA-AES256-SHA384:ECDHE-RSA-AES256-SHA:ECDHE-ECDSA-AES256-SHA:AES128-GCM-SHA256:AES256-GCM-SHA384:ECDHE-RSA-RC4-SHA:ECDHE-ECDSA-RC4-SHA:AES128:AES256:RC4-SHA:HIGH:!aNULL:!eNULL:!EXPORT:!DES:!3DES:!MD5:!PSK:!DH
-    SSLHonorCipherOrder on
     SSLCertificateFile <%= @etherpad_ssl_cert %>
     SSLCertificateKeyFile <%= @etherpad_ssl_key %>
     SSLCACertificatePath /etc/ssl/certs
+    <%= @ssl_settings.join("\n") %>
 
     RewriteEngine on
     ProxyVia On

-- 
To view, visit https://gerrit.wikimedia.org/r/153978
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Id35d5235d2ac3141984f3fa6e5991f15c4f00030
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Dzahn <[email protected]>
Gerrit-Reviewer: Alexandros Kosiaris <[email protected]>
Gerrit-Reviewer: Dzahn <[email protected]>
Gerrit-Reviewer: JanZerebecki <[email protected]>
Gerrit-Reviewer: Matanya <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to