Code review comment for lp:~lathiat/charm-helpers/lp1484489-ssl-certificate-chain

Revision history for this message
Jorge Niedbalski (niedbalski) wrote :

Hello Trent,

On the description you mention that this new config declaration only works on <= Trusty, in Xenial the SSLCertificateChainFile option has been deprecated[0].

Do you know if the config parser will fail in >= Xenial if the option is present? if that's the case it might be required to split the template or add a conditional to check the series.

[0] https://httpd.apache.org/docs/current/mod/mod_ssl.html#page-header

review: Needs Information

« Back to merge proposal