[pbs-devel] [PATCH proxmox-backup v2 1/1] fix: s3: make s3_refresh apihandler sync

Nicolas Frey n.frey at proxmox.com
Wed Jan 7 13:46:04 CET 2026


fixes regression from 524cf1e7 that made `datastore::s3_refresh` sync
but did not change the ApiHandler matching part here

This would result in a panic every time an s3-refresh was initiated

Reviewed-by: Christian Ebner <c.ebner at proxmox.com>
Tested-by: Christian Ebner <c.ebner at proxmox.com>
Reviewed-by: Samuel Rufinatscha <s.rufinatscha at proxmox.com>
Fixes: 524cf1e7 ("api: admin: make s3 refresh handler sync") 
Fixes: https://forum.proxmox.com/threads/178655
Signed-off-by: Nicolas Frey <n.frey at proxmox.com>
---
added Fixes trailer to reference blamed commit

 src/bin/proxmox_backup_manager/datastore.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/bin/proxmox_backup_manager/datastore.rs b/src/bin/proxmox_backup_manager/datastore.rs
index 57b4ca29..5c65c5ec 100644
--- a/src/bin/proxmox_backup_manager/datastore.rs
+++ b/src/bin/proxmox_backup_manager/datastore.rs
@@ -339,7 +339,7 @@ async fn s3_refresh(mut param: Value, rpcenv: &mut dyn RpcEnvironment) -> Result
 
     let info = &api2::admin::datastore::API_METHOD_S3_REFRESH;
     let result = match info.handler {
-        ApiHandler::Async(handler) => (handler)(param, info, rpcenv).await?,
+        ApiHandler::Sync(handler) => (handler)(param, info, rpcenv)?,
         _ => unreachable!(),
     };
 
-- 
2.47.3




More information about the pbs-devel mailing list