all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* [PATCH proxmox-acme v2] acme: fix USER_AGENT variable interpolation in curl calls
@ 2026-08-12 12:02 Aaron Carson
  2026-11-05 13:30 ` applied: " Fabian Grünbichler
  0 siblings, 1 reply; 2+ messages in thread
From: Aaron Carson @ 2026-08-12 12:02 UTC (permalink / raw)
  To: pve-devel; +Cc: aaron

The variable was not interpolated, so ACME requests were sent with
the literal string USER_AGENT as the user agent instead of the
configured value.

Signed-off-by: Aaron Carson <aaron@aaroncarson.co.uk>
---
Hi folks, resending with the correct subject prefix (v1 went out as
[PATCH acme]). Noticed the user agent my DNS server was seeing when
updating records for ACME was literally the string USER_AGENT - the
variable isn't interpolated in the two _get_impl curl calls. My CLA
is on file (office@ responded on 6th Aug).

v2: correcting subject prefix, no code changes

 src/proxmox-acme | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/proxmox-acme b/src/proxmox-acme
index 705f7df..39c770b 100644
--- a/src/proxmox-acme
+++ b/src/proxmox-acme
@@ -348,9 +348,9 @@ _get_impl() {
     _CURL="$_CURL --connect-timeout $t"
   fi
   if [ "$onlyheader" ]; then
-    $_CURL -I --user-agent "USER_AGENT" -H "$_H1" -H "$_H2" -H "$_H3" -H "$_H4" -H "$_H5" "$url"
+    $_CURL -I --user-agent "$USER_AGENT" -H "$_H1" -H "$_H2" -H "$_H3" -H "$_H4" -H "$_H5" "$url"
   else
-    $_CURL --user-agent "USER_AGENT" -H "$_H1" -H "$_H2" -H "$_H3" -H "$_H4" -H "$_H5" "$url"
+    $_CURL --user-agent "$USER_AGENT" -H "$_H1" -H "$_H2" -H "$_H3" -H "$_H4" -H "$_H5" "$url"
   fi
   ret=$?
   if [ "$ret" != "0" ]; then
-- 
2.52.0




^ permalink raw reply related	[flat|nested] 2+ messages in thread

* applied: [PATCH proxmox-acme v2] acme: fix USER_AGENT variable interpolation in curl calls
  2026-08-12 12:02 [PATCH proxmox-acme v2] acme: fix USER_AGENT variable interpolation in curl calls Aaron Carson
@ 2026-11-05 13:30 ` Fabian Grünbichler
  0 siblings, 0 replies; 2+ messages in thread
From: Fabian Grünbichler @ 2026-11-05 13:30 UTC (permalink / raw)
  To: pve-devel, Aaron Carson


On Wed, 12 Aug 2026 12:02:28 +0000, Aaron Carson wrote:
> The variable was not interpolated, so ACME requests were sent with
> the literal string USER_AGENT as the user agent instead of the
> configured value.
> 
> 

Applied, thanks!

[1/1] acme: fix USER_AGENT variable interpolation in curl calls
      commit: ca6249a55d6ae61f703e98eb36e8f49acff27084

Best regards,
-- 
Fabian Grünbichler <f.gruenbichler@proxmox.com>




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-11-05 13:32 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-08-12 12:02 [PATCH proxmox-acme v2] acme: fix USER_AGENT variable interpolation in curl calls Aaron Carson
2026-11-05 13:30 ` applied: " Fabian Grünbichler

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal