]> git.proxmox.com Git - pmg-api.git/commit
api: nodeconfig: validate acme config before writing
authorStoiko Ivanov <s.ivanov@proxmox.com>
Wed, 16 Jun 2021 18:36:40 +0000 (20:36 +0200)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Mon, 28 Jun 2021 13:19:40 +0000 (15:19 +0200)
commit08d71f69082b947fc6da756cbe3a40d4c0cafac6
tree162d6ac4e2412e83c3361b93dfba421ba030bb9f
parentb8b56fbfa51d98c31f9ea10124ac3b3c7b68a711
api: nodeconfig: validate acme config before writing

Currently it is possible to add the same domains as different
acmedomainX keys to the node config, which prevents the user from
ordering certificates later.

This patch adds a call to get_acme_conf, which does the semantic
validation (and is also used in all other sites, which read the
config).

Reported in our community forum:
https://forum.proxmox.com/threads/lets-encrypt-cert-on-gui-not-working.91014/

quickly tested in my setup, by successfully adding the same domain
twice without the patch, and failing to do so with it applied.

Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>
src/PMG/API2/NodeConfig.pm