From 0db4d9031bc4c3382add672bf91c3f0c5db8d069 Mon Sep 17 00:00:00 2001 From: Christian Ebner Date: Tue, 26 Nov 2024 13:24:19 +0100 Subject: [PATCH] api types: add missing conf to blob archive name mapping Commit addfae26 ("api types: introduce `BackupArchiveName` type") introduced a dedicated archive name api type to add rust type checking and bundle helpers to the api type. Since this, the backup archive name to server archive name mapping is handled by its parser. This however did not cover the `.conf` extension used for VM config files. Add the missing `.conf` to `.conf.blob` to the match statement and the test cases. Fixes: addfae26 ("api types: introduce `BackupArchiveName` type") Reported-by: Stoiko Ivanov Signed-off-by: Christian Ebner --- pbs-api-types/src/datastore.rs | 3 +++ 1 file changed, 3 insertions(+) diff --git a/pbs-api-types/src/datastore.rs b/pbs-api-types/src/datastore.rs index 90f1195b..d3876838 100644 --- a/pbs-api-types/src/datastore.rs +++ b/pbs-api-types/src/datastore.rs @@ -1841,6 +1841,7 @@ impl BackupArchiveName { Some("ppxar") => ArchiveType::DynamicIndex, Some("pcat1") => ArchiveType::DynamicIndex, Some("img") => ArchiveType::FixedIndex, + Some("conf") => ArchiveType::Blob, Some("json") => ArchiveType::Blob, Some("key") => ArchiveType::Blob, Some("log") => ArchiveType::Blob, @@ -1918,6 +1919,8 @@ mod tests { "/valid/rsa-encrypted.key.blob", "/valid/archive-name.log", "/valid/archive-name.log.blob", + "/valid/qemu-server.conf", + "/valid/qemu-server.conf.blob", ]; for archive_name in valid_archive_names {