Page MenuHomePhabricator

Stop using SSLCertificateChainFile on RSA+EC setups
Closed, ResolvedPublic

Description

As noticed while debugging T369008 we have some httpd misconfigured across the entire fleet. As already noted in Apache documentation SSLCertificateChainFile won't work on dual stack setups:

But be careful: Providing the certificate chain works only if you are using a single RSA or DSA based server certificate. If you are using a coupled RSA+DSA certificate pair, this will work only if actually both certificates use the same certificate chain. Else the browsers will be confused in this situation.

a quick git grep shows the following offenders:

  • icinga
  • karma
  • klaxon
  • librenms
  • mirrors
  • orchestrator
  • gerrit

SSLCertificateChainFile+ SSLCertificateFile combo should be dropped in favor of a single SSLCertificateFile pointing to the chained flavor of the crt:

SSLCertificateFile /etc/acmecerts/${'cert_name'}/live/ec-prime256v1.chained.crt
SSLCertificateFile /etc/acmecerts/${'cert_name'}/live/rsa-2048.chained.crt

Event Timeline

Change #1051301 had a related patch set uploaded (by Filippo Giunchedi; author: Filippo Giunchedi):

[operations/puppet@production] o11y: serve LE chained certificates

https://gerrit.wikimedia.org/r/1051301

Change #1051304 had a related patch set uploaded (by Vgutierrez; author: Vgutierrez):

[operations/puppet@production] gerrit: Stop using SSLCertificateChainFile

https://gerrit.wikimedia.org/r/1051304

Change #1051301 merged by Filippo Giunchedi:

[operations/puppet@production] o11y: serve LE chained certificates

https://gerrit.wikimedia.org/r/1051301

Change #1051307 had a related patch set uploaded (by Vgutierrez; author: Vgutierrez):

[operations/puppet@production] mirrors: Stop using SSLCertificateChainFile

https://gerrit.wikimedia.org/r/1051307

Change #1051310 had a related patch set uploaded (by Vgutierrez; author: Vgutierrez):

[operations/puppet@production] orchestrator: Stop using SSLCertificateChainFile

https://gerrit.wikimedia.org/r/1051310

Change #1051310 merged by Vgutierrez:

[operations/puppet@production] orchestrator: Stop using SSLCertificateChainFile

https://gerrit.wikimedia.org/r/1051310

Change #1051307 merged by Vgutierrez:

[operations/puppet@production] mirrors: Stop using SSLCertificateChainFile

https://gerrit.wikimedia.org/r/1051307

Orchestrator looking fine after a reload of its apache

Change #1051304 merged by Vgutierrez:

[operations/puppet@production] gerrit: Stop using SSLCertificateChainFile

https://gerrit.wikimedia.org/r/1051304

Vgutierrez updated the task description. (Show Details)