[pve-devel] [PATCH-SERIES qemu-server/storage/common 0/4] fix UTF-8 handling for PBS_PASSWORD
Fiona Ebner
f.ebner at proxmox.com
Wed Oct 1 12:47:09 CEST 2025
The PBS password is saved as UTF-8 and decoded to Perl's internal
string representation upon reading from the password file. When the
password contains multi-byte UTF-8 characters, backing up a diskless
VM would fail with:
> Error: error building client for repository XXX -
> PBS_PASSWORD contains bad characters
The same error would occur for uploading the log file after backup, as
well as extracting the configuration file from backup.
Restoring would fail with:
> restore failed: invalid utf-8 sequence of 1 bytes from index 0
Fix these issues by properly encoding the value for the PBS_PASSWORD
environment variable value again as UTF-8.
For PMG, using passwords that would be auto-encoded by Perl as either
ASCII or UTF-8 already worked, but other encodings would not, for
example ISO-8859 would result in:
> proxmox-backup-client failed: Error: error building client for
> repository latin at pbs@10.10.100.180:8007:bigone - PBS_PASSWORD
> contains bad characters (500)
Follow pve-storage commit 5245e04 ("fix #5181: pbs: store and read
passwords as unicode") and align the behavior of the storage plugin
and pbs client module.
qemu-server:
Fiona Ebner (2):
qmp client: encode JSON as UTF-8 to fix PBS backup when password
contains multi-byte UTF-8
pbs: properly encode PBS password as UTF-8 when setting the
environment variable
src/PVE/QMPClient.pm | 14 ++++++++------
src/PVE/QemuServer.pm | 4 ++++
src/PVE/VZDump/QemuServer.pm | 4 ++++
3 files changed, 16 insertions(+), 6 deletions(-)
storage:
Fiona Ebner (1):
pbs plugin: raw client command: properly encode PBS password as UTF-8
when setting the environment variable
src/PVE/Storage/PBSPlugin.pm | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
common:
Fiona Ebner (1):
pbs client: allow using password that would be auto-encoded as neither
ASCII nor UTF-8
src/PVE/PBSClient.pm | 13 ++++++++++---
1 file changed, 10 insertions(+), 3 deletions(-)
Summary over all repositories:
5 files changed, 32 insertions(+), 11 deletions(-)
--
Generated by git-murpp 0.5.0
More information about the pve-devel
mailing list