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.

Sign up for GitHub

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

Jump to bottom

Scrubs email addresses as well as passwords #47

Merged
merged 3 commits into from
Oct 24, 2018
Merged

Scrubs email addresses as well as passwords #47

merged 3 commits into from
Oct 24, 2018

Conversation

ChaosExAnima
Copy link
Contributor

Resolves #46.

Note I have not tested this yet.

@ChaosExAnima ChaosExAnima self-assigned this Oct 11, 2018
@ChaosExAnima
Copy link
Contributor Author

Okay, performed tests:

  1. Created new WP installation and generated 100 users.
  2. Created a snapshot using wpsnapshots create.
  3. Pulled in snapshot.

Results were that users had scrubbed emails ([email protected]) instead of old emails. Passwords were changed as expected.

@tlovett1
Copy link
Member

@ChaosExAnima can you rebase against master please?

@ChaosExAnima
Copy link
Contributor Author

@tlovett1 - Done!

@tlovett1
Copy link
Member

Awesome. Nice work.

@tlovett1 tlovett1 merged commit b15373d into master Oct 24, 2018
@ChaosExAnima ChaosExAnima deleted the feature/scrub-emails branch October 24, 2018 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tlovett1 tlovett1 Awaiting requested review from tlovett1

@cmmarslender cmmarslender Awaiting requested review from cmmarslender

Assignees

@ChaosExAnima ChaosExAnima

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ChaosExAnima @tlovett1