-
Notifications
You must be signed in to change notification settings - Fork 156
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 ability to re-encrypt secrets #456
Comments
@jeffpaul Ping me here or in PR's This needs to be done in batches otherwise you might take down the DB. |
@calvinalkan thanks for reaching out on this, if you've got any other technical direction then please share that here (or if you're open to crafting a PR to ensure this is done in batches then please do so and I'll coordinate on getting that reviewed/merged). |
Actually, I was thinking it might be better to force reset everything.. An inconvenience is better than a hack. |
Our library snicco/better-wpdb now has support for batch processing with db locks. The prerequisite is PHP7.4 |
If we take the approach that I outlined in #389 (comment), then I think the best approach is to give admins these two options to choose from:
|
Add a WP CLI command or plugin setting to re-encrypt secrets for all users by supplying the old key and the new key.
More details to be shared.
The text was updated successfully, but these errors were encountered: