[pve-devel] [PATCH common 1/2] REST environment: warn helpers: use warn instead of printing to stderr
Fiona Ebner
f.ebner at proxmox.com
Mon Feb 5 13:28:53 CET 2024
Like this, __WARN__ handlers will still be called. In particular,
daemons like pvestatd will set a __WARN__ handler and also log
warnings to syslog. The intention behind introducing log_warn() was to
make warnings more visible, not less, so fix the semantics to make
sure switching from warn to log_warn() does not have this unintended
side-effect.
Reported-by: Friedrich Weber <f.weber at proxmox.com>
Signed-off-by: Fiona Ebner <f.ebner at proxmox.com>
---
src/PVE/RESTEnvironment.pm | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/PVE/RESTEnvironment.pm b/src/PVE/RESTEnvironment.pm
index 191c6eb..41efb16 100644
--- a/src/PVE/RESTEnvironment.pm
+++ b/src/PVE/RESTEnvironment.pm
@@ -723,7 +723,7 @@ sub log_warn {
$rest_env->warn($message);
} else {
chomp($message);
- print STDERR "WARN: $message\n";
+ warn "WARN: $message\n";
}
}
@@ -732,7 +732,7 @@ sub warn {
chomp($message);
- print STDERR "WARN: $message\n";
+ warn "WARN: $message\n";
$self->{warning_count}++;
}
--
2.39.2
More information about the pve-devel
mailing list