public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
* [pve-devel] applied: [PATCH manager] fix #2890: also detect & renew DNS-only ACME certs
@ 2020-07-27  8:55 Fabian Grünbichler
  0 siblings, 0 replies; only message in thread
From: Fabian Grünbichler @ 2020-07-27  8:55 UTC (permalink / raw)
  To: pve-devel

this call site was apparently missed when we refactored the node config
/ ACME interaction.

Suggested/Reported-by: Frédéric Bourqui
Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>
---
 bin/pveupdate | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/bin/pveupdate b/bin/pveupdate
index 56d5d718..99b52fe9 100755
--- a/bin/pveupdate
+++ b/bin/pveupdate
@@ -60,7 +60,8 @@ if (my $err = $@) {
 
 eval {
     my $node_config = PVE::NodeConfig::load_config($nodename);
-    if ($node_config && $node_config->{acme}) {
+    my $acme_node_config = PVE::NodeConfig::get_acme_conf($node_config);
+    if ($acme_node_config && $acme_node_config->{domains}) {
 	my $cert = PVE::CertHelpers::cert_path_prefix($nodename).".pem";
 	if (-e $cert) {
 	    if (PVE::Certificate::check_expiry($cert, time() + 30*24*60*60)) {
-- 
2.20.1





^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2020-07-27  8:56 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-27  8:55 [pve-devel] applied: [PATCH manager] fix #2890: also detect & renew DNS-only ACME certs Fabian Grünbichler

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal