my $code = sub {
my $cfg = PMG::Config->new();
- if (!$cfg->get('mail', 'tls') == !$on) {
- return;
+ if (!$cfg->get('mail', 'tls') != !$on) {
+ print "Rewriting postfix config\n";
+ $cfg->set('mail', 'tls', $on);
+ $cfg->write();
+ my $changed = $cfg->rewrite_config_postfix();
}
- print "Rewriting postfix config\n";
- $cfg->set('mail', 'tls', $on);
- $cfg->write();
- my $changed = $cfg->rewrite_config_postfix();
-
- if ($changed && $reload) {
+ if ($reload) {
print "Reloading postfix\n";
PMG::Utils::service_cmd('postfix', 'reload');
}
if (defined($old_cert)) {
print "Revoking old certificate\n";
- eval { $acme->revoke_certificate($old_cert, undef) };
+ eval {
+ $old_cert = pem_certificate($old_cert)
+ or die "no certificate section found in '$cert_path'\n";
+ $acme->revoke_certificate($old_cert, undef);
+ };
warn "Revoke request to CA failed: $@" if $@;
}
};