[pve-devel] [PATCH pve-container 1/2] pct: set worker user for pull_file/push_file calls

Oguz Bektas o.bektas at proxmox.com
Mon Mar 7 10:20:42 CET 2022


was previously unset, causing a 'root at pve' to show up in the task logs
instead of the regular 'root at pam'.

Signed-off-by: Oguz Bektas <o.bektas at proxmox.com>
---
 src/PVE/CLI/pct.pm | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/PVE/CLI/pct.pm b/src/PVE/CLI/pct.pm
index 462917b..99c160c 100755
--- a/src/PVE/CLI/pct.pm
+++ b/src/PVE/CLI/pct.pm
@@ -544,6 +544,8 @@ __PACKAGE__->register_method({
 
 	my $rpcenv = PVE::RPCEnvironment::get();
 
+	my $authuser = $rpcenv->get_user();
+
 	my $vmid = extract_param($param, 'vmid');
 	my $path = extract_param($param, 'path');
 	my $dest = extract_param($param, 'destination');
@@ -578,7 +580,7 @@ __PACKAGE__->register_method({
 	    };
 
 	    # This avoids having to setns() back to our namespace.
-	    return $rpcenv->fork_worker('pull_file', $vmid, undef, $realcmd);
+	    return $rpcenv->fork_worker('pull_file', $vmid, $authuser, $realcmd);
 	};
 
 	return PVE::LXC::Config->lock_config($vmid, $code);
@@ -627,6 +629,8 @@ __PACKAGE__->register_method({
 
 	my $rpcenv = PVE::RPCEnvironment::get();
 
+	my $authuser = $rpcenv->get_user();
+
 	my $vmid = extract_param($param, 'vmid');
 	my $file = extract_param($param, 'file');
 	my $dest = extract_param($param, 'destination');
@@ -682,7 +686,7 @@ __PACKAGE__->register_method({
 	    };
 
 	    # This avoids having to setns() back to our namespace.
-	    return $rpcenv->fork_worker('push_file', $vmid, undef, $realcmd);
+	    return $rpcenv->fork_worker('push_file', $vmid, $authuser, $realcmd);
 	};
 
 	return PVE::LXC::Config->lock_config($vmid, $code);
-- 
2.30.2






More information about the pve-devel mailing list