nextcloud / server

☁️ Nextcloud server, a safe home for all your data
https://nextcloud.com
GNU Affero General Public License v3.0
27.57k stars 4.08k forks source link

Allow for filtering log messages #38403

Open magikmw opened 1 year ago

magikmw commented 1 year ago

How to use GitHub

Is your feature request related to a problem? Please describe. A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Since LDAP spews out errors every time it cannot sync disabled accounts, my log is constantly spammed, and essentially unusable via the web view for ERROR level specifically. Any syslog has to filter those out as well. There's probably other warnings or errors I don't care about, either due to faulty app reporting or otherwise.

Describe the solution you'd like A clear and concise description of what you want to happen.

I would like a way to filter out certain log messages from appearing in both text logfile / syslog and web log view. A simple regular expression setting for audit_admin would suffice.

Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.

No solutions I know of that wouldn't require fixing issues with 3rd party or built-in apps, therefore reducing log spam. Syslog can be filtered on the receiving end, but it's hardly ideal.

Additional context Add any other context or screenshots about the feature request here.

joshtrichards commented 1 year ago

The admin_audit log provides special event logging, but isn't what provides the web interface under Administration settings->Logging. It's also not the source/handler for error log entries generally. Are you sure you're referring to the admin_audit log and not general logging? Can you provide some example log entries?

Also, it sounds like one of the root issues is perhaps some logging adjustments within the user_ldap app being needed. Have you opened a separate Issue on that? Maybe it'll makes sense to shift the logging for disabled account handling down to INFO level or something so it doesn't show up by default.

magikmw commented 1 year ago

Thanks for the response. I might be confused about the module reaponsible for logs I mentioned in OP. I do indeed mean general logs, just thought it's admin_audit that's handling them. Perhaps title requires changing.

I will update with particular examples when I'm able. From what I've seen, user_ldap is ment to produce errors, but it may just be some kind of confusion as well. Will update as soon as I can.

Nonetheless, I think a generic filtering feature for logs in Admin -> Logging, and nextcloud.log would be great. There's a bunch of unofficial apps with different amounts of support, and the only alternative to having a workable and high information to noise ratio logs is disabling the noisy apps.