Admin feature: Nuke user #2977
Admin feature: Nuke user #2977
Conversation
| for project in user.projects: | ||
| remove_project(project, request, flash=False) | ||
|
|
||
| request.db.delete(user) |
There was a problem hiding this comment.
this will implicitly cascade to their associated emails?
There was a problem hiding this comment.
also probably want a JournalEntry
There was a problem hiding this comment.
Yes, this will delete emails and roles for that user.
I thought about adding a JournalEntry here, but it seems like they're project-specific? E.g., we don't make them when a user account is created, just when stuff changes with a specific project.
There was a problem hiding this comment.
fair, but I would like some record of this feature being used. the journal is freeform. we should just write something to note that an admin used this feature.
There was a problem hiding this comment.
JournalEntry(
action='purge_user',
name=f'user:{user.username}',
submitted_by=request.user,
submitted_from=request.remote_addr,
)
would work and can't collide with project names.
warehouse/admin/views/users.py
Outdated
|
|
||
| # Delete projects one by one so they are purged from the cache | ||
| for project in user.projects: | ||
| remove_project(project, request, flash=False) |
There was a problem hiding this comment.
There's already a JournalEntry happening in remove_project.
There was a problem hiding this comment.
confirmed, sorry I tested the button locally on a user with no projects :)
Towards fixing #2976.
Remove the user account, any projects they own, associated releases, roles, emails, etc.