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

[Christian Ebner <c.ebner@proxmox.com>]

Patch looks good to me and restores the correct behavior of the cli command.

On 1/5/26 11:34 AM, Nicolas Frey wrote:
> fixes regression from 524cf1e 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
> 
> Fixes: https://forum.proxmox.com/threads/178655

Might reference the blamed commit in a fixes trailer as well for easier 
lookup and search:

Fixes: 524cf1e7 ("api: admin: make s3 refresh handler sync")

> Signed-off-by: Nicolas Frey <n.frey@proxmox.com>
> ---
>   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!(),
>       };
>   

Reviewed-by: Christian Ebner <c.ebner@proxmox.com>
Tested-by: Christian Ebner <c.ebner@proxmox.com>


_______________________________________________
pbs-devel mailing list
pbs-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel