[pve-devel] [PATCH common/cluster 0/3] fix #4937: fix utf8 encoding issues while saving notification config

Lukas Wagner l.wagner at proxmox.com
Wed Aug 30 14:37:51 CEST 2023


These patches should fix issues with certain special characters
(e.g ü) in the notification configuration [1].
Before, when setting a comment for an endpoint to certain values 
(e.g. 'für admins'), the resulting saved configuration would 
contain invalid UTF8 text.

The reason for that was the configuration was returned from
Rust as a proper utf8 string, however it was saved to the file 
system without specifying an encoding.


[1] https://bugzilla.proxmox.com/show_bug.cgi?id=4937



pve-common:

Lukas Wagner (1):
  tools: allow to specify file encoding for file_set_contents

 src/PVE/Tools.pm | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)


pve-cluster:

Lukas Wagner (2):
  cluster fs: allow to specify file encoding for cfs_write_file
  fix #4937: notify: write configuration files in utf8 encoding

 src/PVE/Cluster.pm | 4 ++--
 src/PVE/Notify.pm  | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)


Summary over all repositories:
  3 files changed, 10 insertions(+), 5 deletions(-)

-- 
murpp v0.4.0






More information about the pve-devel mailing list