Skip to content

Conversation

@tomako
Copy link
Contributor

@tomako tomako commented Feb 26, 2025

This PR is about adding UserLogEntry for logging events related to user records.

Highlights:

  • The Logs menu is placed in User Profile side-menu and visible only for admins
  • In this PR only the user creation is logged
  • The user log records are deleted during user anonymization

@github-actions github-actions bot added the alembic Contains database changes label Feb 26, 2025
@tomako tomako force-pushed the add_user_log_entry branch from 1dabb77 to 9051e31 Compare February 26, 2025 23:26
@tomako tomako force-pushed the add_user_log_entry branch from 9051e31 to e529dc5 Compare February 27, 2025 00:20
@tomako tomako requested a review from ThiefMaster February 27, 2025 00:25
@tomako tomako force-pushed the add_user_log_entry branch 2 times, most recently from d417b58 to 32d56c6 Compare March 13, 2025 22:36
@tomako
Copy link
Contributor Author

tomako commented Mar 14, 2025

@ThiefMaster What do you think about logging ACL changes on both object and user level?
That would mean some changes here and also adding other signal handler(s) for objects whose don't have own logs.

@ThiefMaster
Copy link
Member

It sounds useful, but something I'd rather have in a separate PR.

@ThiefMaster ThiefMaster added this to the v3.3 milestone Mar 20, 2025
@ThiefMaster ThiefMaster merged commit 2b6d449 into indico:master Mar 21, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

alembic Contains database changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants