From: "Fabian Grünbichler" <f.gruenbichler@proxmox.com>
To: Mathias Bartmann <mathias.bartmann@gmail.com>,
pve-devel@lists.proxmox.com
Subject: applied-series: [PATCH proxmox-acme 0/3] acme: fix missing/diverging shim helpers, tighten missing-function check
Date: Thu, 05 Nov 2026 14:54:18 +0100 [thread overview]
Message-ID: <1793886673.34ngzwnpaw.astroid@yuna.none> (raw)
In-Reply-To: <20260925124737.42866-1-mathias.bartmann@gmail.com>
thanks!
On September 25, 2026 2:47 pm, Mathias Bartmann wrote:
> dns_oci.sh (Oracle Cloud DNS) cannot complete a DNS-01 challenge,
> see bug #7851 for the full analysis. Patch 1 fixes the two shim helpers
> behind it and adds a test running them through the shim.
>
> While looking at why this was not caught at build time, it turned out
> that check-missing-functions misses calls at the end of a command
> substitution or pipe, and drops any line that also has an assignment.
> With that fixed (patch 3) it finds a third missing helper, _url_replace
> used by dns_yc.sh, added in patch 2 so the series passes at every
> commit.
>
> A minimal fix within the existing grep pipeline is possible, but it
> still drops any line that also calls a known helper, so it would not
> catch _url_replace, and it needs ten more false-positive entries for
> variables after export, unset and read. The rewrite checks each called
> name individually, so the expected list shrinks to the one real
> upstream issue (dns_aws.sh: _error). Happy to send the minimal variant
> instead if you prefer.
>
> Patch 1 was verified end to end on PVE 9.2 with an OCI-hosted zone:
> certificate ordered and installed on pveproxy. Patch 2 is a verbatim
> copy of the upstream helper, but I could not test it against Yandex
> Cloud.
>
> Mathias Bartmann (3):
> fix #7851: acme: add missing _mktemp and restore _dbase64 default
> acme: add missing _url_replace function
> tests: rewrite check-missing-functions to catch calls in substitutions
>
> src/proxmox-acme | 15 +++++++-
> src/test/Makefile | 2 +-
> src/test/check-missing-functions | 59 ++++++++++++++++++++++++-----
> src/test/missing-functions.expected | 8 +---
> src/test/test-shim-helpers | 37 ++++++++++++++++++
> 5 files changed, 102 insertions(+), 19 deletions(-)
> create mode 100755 src/test/test-shim-helpers
>
> --
> 2.55.0
>
>
>
>
>
prev parent reply other threads:[~2026-11-05 13:54 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-25 12:47 [PATCH proxmox-acme 0/3] acme: fix missing/diverging shim helpers, tighten missing-function check Mathias Bartmann
2026-09-25 12:47 ` [PATCH proxmox-acme 1/3] fix #7851: acme: add missing _mktemp and restore _dbase64 default Mathias Bartmann
2026-09-25 12:47 ` [PATCH proxmox-acme 2/3] acme: add missing _url_replace function Mathias Bartmann
2026-09-25 12:47 ` [PATCH proxmox-acme 3/3] tests: rewrite check-missing-functions to catch calls in substitutions Mathias Bartmann
2026-11-05 13:54 ` Fabian Grünbichler [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1793886673.34ngzwnpaw.astroid@yuna.none \
--to=f.gruenbichler@proxmox.com \
--cc=mathias.bartmann@gmail.com \
--cc=pve-devel@lists.proxmox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox