Skip to content
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

Notify superusers based on records in action log #2134

Closed
pdurbin opened this issue May 5, 2015 · 4 comments
Closed

Notify superusers based on records in action log #2134

pdurbin opened this issue May 5, 2015 · 4 comments

Comments

@pdurbin
Copy link
Member

pdurbin commented May 5, 2015

Original suggestion:

"Have a record in the audit log of the requested password resets with the system notifying the administrator of suspicious behavior when there appears to be an attack going on." -- @eaquigley at #2030 (comment)

This is a great idea but needs further clarification and design.

  • The action log is quite chatty. How much of it should we expose to superusers via notifications?

Also, only "commands" are logged to the action log so we would have to convert password reset to use the Command pattern. This can be done but is non-trivial.

@pdurbin
Copy link
Member Author

pdurbin commented Feb 3, 2016

Also, only "commands" are logged to the action log so we would have to convert password reset to use the Command pattern.

@scolapasta explained that this is not the case. Password reset actions, even though they don't use the Command pattern, are logged. You can find them with select * from actionlogrecord where actiontype = 'BuiltinUser'; as I mentioned at #2879 (comment)

@pdurbin
Copy link
Member Author

pdurbin commented Jun 15, 2016

@bmckinney you seem interested in making good use of the actionlogrecord table. Do you or @pameyer have opinions on this issue? Is it something you'd like to tackle? @pameyer and I chatted a bit about this at http://irclog.iq.harvard.edu/dataverse/2016-06-14#i_36923

@pdurbin
Copy link
Member Author

pdurbin commented Dec 21, 2016

This seem related to #2130 and I wonder these notifications would make sense to add to the dashboard being designed at https://trello.com/c/kDSgIVVS/8-administrative-dashboard-4-7 as part of #840.

@pdurbin
Copy link
Member Author

pdurbin commented Jun 28, 2017

I'm talking to myself in this issue. 😄 If anyone wants this, please let me know. Closing.

@pdurbin pdurbin closed this as completed Jun 28, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants