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

[Lukas Wagner]

On 8/30/23 14:37, Lukas Wagner wrote:
> 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

Since I'm on vacation for the next two weeks:
These changes should be applied before the new notification system hits 
the no-subscription repo (pve-manager 8.0.5), since this bug can break
notifications entirely (to fix it, one has to delete notifications.cfg 
and priv/notifications.cfg, or at least the faulty characters from these 
files)

-- 
- Lukas