Skip to content

Conversation

@brianhelba
Copy link
Collaborator

These were added by #59, to migrate users from Girder to Django.

@brianhelba
Copy link
Collaborator Author

Looking at the production DANDI database, there are only 7 remaining placeholder users:

select * from auth_user where email LIKE 'placeholder_%';

Since the users need to login to convert their placeholder account, I believe none of the 7 have logged in since the migration was added in April 2021. Could we just delete the 7 user accounts in question?

@mvandenburgh
Copy link
Member

Unfortunately, some of those users are owners of dandisets. For example, https://dandiarchive.org/dandiset/000004 contains one of those placeholder users as an owner.

@brianhelba
Copy link
Collaborator Author

Could we keep that one (or all) of the users in the database, but fix their email addresses and give them a totally random password? That will keep referential integrity in the database.

If those users ever come back after 5 years, they can just do a reset password operation (or sign in with GitHub, if it returns the same email).

I don't know how reset password actually works on DANDI, but I am aware that some of the active users don't have linked social accounts, so they must be logging in somehow.

@jjnesbitt
Copy link
Member

I don't know how reset password actually works on DANDI, but I am aware that some of the active users don't have linked social accounts, so they must be logging in somehow.

I think all of those users are site staff/admins. The login page only gives the option for Github.

Since this is a very small number of users, and they probably have no intention of logging into dandi anytime soon (if ever, since they've not logged in since 2021), I wonder if we can simply send all of these users an email, stating something along the lines of "Please login or your account will be deleted", giving some time-frame like 2 weeks or so. As long as those users aren't the sole owner of a dandiset, then it will be relatively straightforward from our end. At least for the dandiset linked above, there is another owner on that dandiset.

These were added by #59, to migrate users from Girder to Django.
@waxlamp waxlamp merged commit cf3ac4d into master Dec 31, 2025
7 checks passed
@waxlamp waxlamp deleted the user-migration branch December 31, 2025 16:22
@dandibot
Copy link
Member

dandibot commented Jan 8, 2026

🚀 PR was released in v0.21.0 🚀

@dandibot dandibot added the released This issue/pull request has been released. label Jan 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released This issue/pull request has been released.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants