From: William Lallemand <wlallemand@haproxy.com>
Date: Wed, 16 Apr 2025 15:06:52 +0000 (+0200)
Subject: BUG/MINOR: acme/cli: fix certificate name in error message
X-Git-Tag: v3.2-dev11~49
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=115653bfc85a4d82d797a15f0db8c49f6eb5481a;p=thirdparty%2Fhaproxy.git

BUG/MINOR: acme/cli: fix certificate name in error message

The acme command had a new parameter so the certificate name is not
correct anymore because args[1] is not the certificate value anymore.
---

diff --git a/src/acme.c b/src/acme.c
index 86ffd5b78..86199288c 100644
--- a/src/acme.c
+++ b/src/acme.c
@@ -1785,18 +1785,18 @@ static int cli_acme_renew_parse(char **args, char *payload, struct appctx *appct
 		return cli_err(appctx, "Can't update: operations on certificates are currently locked!\n");
 
 	if ((store = ckchs_lookup(args[2])) == NULL) {
-		memprintf(&err, "Can't find the certificate '%s'.\n", args[1]);
+		memprintf(&err, "Can't find the certificate '%s'.\n", args[2]);
 		goto err;
 	}
 
 	if (store->conf.acme.id == NULL) {
-		memprintf(&err, "No ACME configuration defined for file '%s'.\n", args[1]);
+		memprintf(&err, "No ACME configuration defined for file '%s'.\n", args[2]);
 		goto err;
 	}
 
 	cfg = get_acme_cfg(store->conf.acme.id);
 	if (!cfg) {
-		memprintf(&err, "No ACME configuration found for file '%s'.\n", args[1]);
+		memprintf(&err, "No ACME configuration found for file '%s'.\n", args[2]);
 		goto err;
 	}