[PATCH] polkitd: add logging for successfully loaded rules

2023-08-01 Thread Michael Trapp
Log each successful load operation of a rule file including the name of the loaded file. Getting the list of active rule files from the logs is an important information in a support case and should not need additional tracing/debugging. --- Built and tested on top of 4b7a5c3. The info is logged dur

Re: [PATCH] polkitd: add logging for successfully loaded rules

2023-07-20 Thread Trapp, Michael
Hi Jan, that’s interesting, I can see the log messages during polkitd startup but not during runtime. I’ve double checked with a request which triggers polkit authorization and I can’t see the logs during runtime. If you prefer to have the logging bound to the ‘—no-debug’ switch, I’m fine with

Re: [PATCH] polkitd: add logging for successfully loaded rules

2023-07-20 Thread Jan Rybar
Hello Michael, Thanks for the patch! I'm not sure it's really safe to write out about every loaded script though, because that will flood the journal with tens of messages on every polkit event. The parsers log every error during loading the files and I believe there are not that many .rules files

[PATCH] polkitd: add logging for successfully loaded rules

2023-07-20 Thread Michael Trapp
Log each successfully loaded rule file. At the moment there is only a summary message wich does not include any further information about the loaded files. --- src/polkitbackend/polkitbackendjsauthority.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/polkitbackend/polkitbackendjsaut