[pdm-devel] [PATCH datacenter-manager 2/2] server: api: task: return upid in status

Dominik Csapak d.csapak at proxmox.com
Wed Jan 29 11:44:50 CET 2025


upid was not in params, since it's an extra parameter, so simply use that

Signed-off-by: Dominik Csapak <d.csapak at proxmox.com>
---
 server/src/api/nodes/tasks.rs | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/server/src/api/nodes/tasks.rs b/server/src/api/nodes/tasks.rs
index 13a713a..01361bd 100644
--- a/server/src/api/nodes/tasks.rs
+++ b/server/src/api/nodes/tasks.rs
@@ -273,18 +273,14 @@ fn stop_task(upid: UPID, rpcenv: &mut dyn RpcEnvironment) -> Result<(), Error> {
     },
 )]
 /// Get task status.
-async fn get_task_status(
-    upid: UPID,
-    param: Value,
-    rpcenv: &mut dyn RpcEnvironment,
-) -> Result<Value, Error> {
+async fn get_task_status(upid: UPID, rpcenv: &mut dyn RpcEnvironment) -> Result<Value, Error> {
     let auth_id: Authid = rpcenv.get_auth_id().unwrap().parse()?;
     check_task_access(&auth_id, &upid)?;
 
     let task_auth_id: Authid = upid.auth_id.parse()?;
 
     let mut result = json!({
-        "upid": param["upid"],
+        "upid": upid.to_string(),
         "node": upid.node,
         "pid": upid.pid,
         "pstart": upid.pstart,
-- 
2.39.5





More information about the pdm-devel mailing list