[pve-devel] applied: [PATCH v3 common] REST environment: 'root at pam' fallback in task logs if no user specified for forked worker

Thomas Lamprecht t.lamprecht at proxmox.com
Wed Mar 16 07:59:16 CET 2022


On 15.03.22 12:52, Oguz Bektas wrote:
> call sites without a defined user parameter (found only push_file and
> pull_file from pve-container) were logging the user as 'root at pve' which
> doesn't exist.
> 
> for clarity, also add a comment that this is only used for the task
> logs.
> 
> Signed-off-by: Oguz Bektas <o.bektas at proxmox.com>
> ---
> v2->v3:
> * drop warning message
> * pass $noerr to get_user() to avoid dead code in case it fails
> 
>  src/PVE/RESTEnvironment.pm | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
>

applied, with slight adaptions to the commit message, thanks!





More information about the pve-devel mailing list