[pve-devel] applied-series: [PATCH-SERIES proxmox{, -mail-forward, -backup}/pve-manager] add proxmox-mail-forward helper binary

Wolfgang Bumiller w.bumiller at proxmox.com
Thu Nov 10 11:58:53 CET 2022


applied remaining patches

On Fri, Oct 21, 2022 at 03:02:43PM +0200, Fiona Ebner wrote:
> written in Rust, and replacing the pvemailforward binary in PVE. Can
> be used in PVE and PBS as well as in a mixed installations of the two.
> 
> To make reading the config files work, it is a setuid binary owned by
> root, but it sets the effective UID to the real UID after reading the
> configs, so parsing and sendmail invocation happen with lower
> privileges again (well, except if the binary was called by root
> directly).
> 
> The .forward file is updated during postinst as currently done in
> pve-manager. proxmox-mail-forward's postinst will not do anything
> when detecting a pvemailforward entry. Instead pve-manager is
> responsible for the switchover in PVE.
> 
> 
> proxmox-mail-forward needs a depenency bump for proxmox-section-config
> for the functionality added by the first patch.
> 
> proxmox-backup recommends proxmox-mail-forward (I felt a Recommends
> is more fitting, but feel free to change it) and pve-manager depends
> on proxmox-mail-forward are part of the series.
> 
> 
> proxmox:
> 
> Fiona Ebner (1):
>   section config: parse additional properties when schema allows it
> 
>  proxmox-section-config/src/lib.rs | 79 ++++++++++++++++++++++++++++++-
>  1 file changed, 78 insertions(+), 1 deletion(-)
> 
> 
> proxmox-mail-forward:
> 
> Fiona Ebner (3):
>   initial commit
>   add Debian packaging
>   d/postinst: register binary in .forward
> 
> 
> proxmox-backup:
> 
> Fiona Ebner (1):
>   fix #4287: d/control: recommend proxmox-mail-forward
> 
>  debian/control | 1 +
>  1 file changed, 1 insertion(+)
> 
> 
> pve-manager:
> 
> Fiona Ebner (4):
>   d/control: depend on proxmox-mail-forward
>   d/postinst: replace pvemailforward with proxmox-mail-forward
>   remove pvemailforward binary
>   d/control: drop ${shlibs:Depends} for pve-manager
> 
>  bin/Makefile             | 11 +++-------
>  bin/pvemailforward.c     | 17 ---------------
>  bin/pvemailforward.pl    | 45 ----------------------------------------
>  debian/control           |  2 +-
>  debian/lintian-overrides |  4 ----
>  debian/postinst          | 10 +++++++--
>  debian/rules             |  2 +-
>  7 files changed, 13 insertions(+), 78 deletions(-)
>  delete mode 100644 bin/pvemailforward.c
>  delete mode 100755 bin/pvemailforward.pl
> 
> -- 
> 2.30.2





More information about the pve-devel mailing list