Proxmox/pve-manager
5
0
Fork 0
Code Pull Requests Releases Activity

api: ACME renew: handle error from revoking old cert gracefully

Browse Source 
this normally just means that the old cert is already expired, we do
not care for that - after all: we got a new (renewed) valid cert
successfully.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
This commit is contained in:
Thomas Lamprecht 2020-08-19 18:47:08 +02:00
parent 96adad96dd
commit 12f089b5de
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
PVE/API2/ACME.pm
View File

@ -301,7 +301,8 @@ __PACKAGE__->register_method ({
die "$@\n" if $@;
print "Revoking old certificate\n";
$acme->revoke_certificate($old_cert);
eval { $acme->revoke_certificate($old_cert) };
warn "Revoke request to CA failed: $@" if $@;
};
return $rpcenv->fork_worker("acmerenew", undef, $authuser, $realcmd);