Opened 4 years ago
Last modified 15 months ago
#1327 new Feature Wish
revisit msg() messagelevels, go away from "M_INFO for everything"
Reported by: | Gert Döring | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | release 2.7 |
Component: | Generic / unclassified | Version: | OpenVPN git master branch (Community Ed) |
Severity: | Not set (select this one, unless your'e a OpenVPN developer) | Keywords: | logging, msg, M_INFO |
Cc: | Selva Nair, stipa, tct |
Description
errlevel.h has a nice collection of D_ flags for "log this only for --verb 3 or higher, and rate-limit" etc. (D_IFCONFIG_POOL()) - so stuff like pool.c shouldn't do
msg(M_INFO, "ifconfig_pool_read(), in='%s'", BSTR(&in));
for "we read this lengthy file, here's line-by-line output" - M_INFO is "1", so nearly always printed...
But we have very many of these, so
- come to agreement what we want to use
- document this
- go through the files one-by-one and use "the right" code
Change History (5)
comment:1 Changed 4 years ago by
Cc: | Selva Nair stipa added |
---|
comment:2 Changed 4 years ago by
Cc: | tct added |
---|
comment:3 Changed 3 years ago by
comment:5 Changed 15 months ago by
Milestone: | release 2.6 → release 2.7 |
---|
Won't happen for 2.6 ("release coming really soon now"). Revisit early in the 2.7 timeframe.
Note: See
TracTickets for help on using
tickets.
The grasp exceeds the reach ..