New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
WIP: Add audit log module #2042
base: 0.x
Are you sure you want to change the base?
Conversation
value = Value | ||
}, | ||
Context | ||
). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All these observers are quite verbose, but I like that it’s very explicit.
observe_edge_insert/2, | ||
observe_edge_delete/2, | ||
observe_media_update_done/2, | ||
observe_rsc_update_done/2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any other notifications you’d like to see in the audit log?
Maybe we could hook into (the results of) |
@mworrell Not yet, because they publish no notifications. 😉 |
It might be an idea to be able to hook into the |
In the module we made it is possible to have an hierarchy of events so you can easily filter on authentication events, resource modification, etc. In our own codebase we have custom events. What I don't have right now is an admin interface. |
See also #2057 for some simple logging of system-events using the current logging module. |
Description
Fix #1529.
Checklist
log_audit
table.#log_audit{}
properties, similar to the e-mail log.