[PATCH pve-manager v3 0/2] fix #8031: write APT proxy config on http_proxy change

[Jonas Theisen <j.theisen@proxmox.com>]

To allow reuse of the same code this patch series factors out
the relevant function inside the APT part of the API and reuses
the function on change of the http_proxy variable in the Cluster API.

This is to ensure a consistent state after a change of the HTTP proxy
without having to run a Refresh / apt update through the PVE tooling.

https://bugzilla.proxmox.com/show_bug.cgi?id=8031

Changes from v2:
 * Refactored check for proxy change for more human-readability
 * Verified regex pattern

Rationale for the regex pattern's \b boundaries is that potential
new parameters could be named http_proxy_port or similar which
would still match the regex pattern without the \b boundaries.
This can be intentional but should be checked if the problem arises.

Tested:
 * Set HTTP Proxy through GUI updates apt proxy config
 * Removal of HTTP proxy through GUI updates config
 * Removal of http_proxy and bwlimit through API updates config

Thanks for the feedback @m.sandoval!

Jonas Theisen (2):
  fix #8031: factor out APT proxy write from update_database call
  fix #8031: write APT proxy config on http_proxy change

 PVE/API2/APT.pm     | 23 ++++++++++++++---------
 PVE/API2/Cluster.pm | 14 ++++++++++++++
 2 files changed, 28 insertions(+), 9 deletions(-)

-- 
2.47.3