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

Add a mechanism allowing project admins to "delete" a user #2445

Merged
merged 1 commit into from
Apr 30, 2018

Conversation

davidpanderson
Copy link
Contributor

This leaves the user record and host records in the DB
to avoid dangling references from the result table.
It:

  • changes the user name, email addr, and auth to "deleted_xxx"
  • removes the user from team
  • removes posts by the user
  • removes PMs by the user
  • removes friend connection
  • removes profile
  • "anonymizes" the hosts (erase IP addr and domain name)

Note 1: this is accessed by admins using a script in ops/;
It's not available to users.

Note 2: this may not satisfy the requirements of EU-GDPR
since it doesn't delete the user and host records.

This leaves the user record and host records in the DB
to avoid dangling references from the result table.
It:
- changes the user name, email addr, and auth to "deleted_xxx"
- removes the user from team
- removes posts by the user
- removes PMs by the user
- removes friend connection
- removes profile
- "anonymizes" the hosts (erase IP addr and domain name)

Note 1: this is accessed by admins using a script in ops/;
It's not available to users.

Note 2: this may not satisfy the requirements of EU-GDPR
since it doesn't delete the user and host records.
@TheAspens
Copy link
Member

David - do you mind if we hold on reviewing this to see how the discussion in #2447 goes? Since there might be a lot of overlap between the two it would be good to have one implementation if that will meet the needs that led to this pull request as well as GDPR?

@davidpanderson
Copy link
Contributor Author

davidpanderson commented Apr 3, 2018 via email

@TheAspens
Copy link
Member

@davidpanderson - as I think through #2447 - what was the driver for this pull request? I.e. I'd like to understand what are the needs that led to the creation of this pull request so that I can understand what other steps might be needed as I implement the other issue.

@RichardHaselgrove
Copy link
Contributor

There was a very strong request by one particular user to be erased from the BOINC-site, and potentially from project sites as well. Obviously I can't identify the user here, but they are known to me personally, and I can brief you privately if you wish.

@TheAspens
Copy link
Member

I don't need the details - just a general idea which you provided. It sounds like whatever we implement for GDPR will have satisified that request as well. Let me know if you agree.

@RichardHaselgrove
Copy link
Contributor

I would agree with that. I don't think the user concerned was consciously requesting a dummy run for GDPR, and I don't think that David was consciously operating GDPR, but in effect that's what happened. If you follow the current path of merging your legal advice with David's PR, the final result should have the best of both.

@TheAspens
Copy link
Member

I'm going to merge this to master and use it in pull request #2472

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants