Skip to content

Improve robustness of WP-CLI commands and 2FA state cleanup - #5

Open
nimesh-xecurify wants to merge 1 commit into
masteradhoc:233-add-wpcli-foundationfrom
nimesh-xecurify:wp-cli-tests-work
Open

Improve robustness of WP-CLI commands and 2FA state cleanup#5
nimesh-xecurify wants to merge 1 commit into
masteradhoc:233-add-wpcli-foundationfrom
nimesh-xecurify:wp-cli-tests-work

Conversation

@nimesh-xecurify

@nimesh-xecurify nimesh-xecurify commented Aug 3, 2026

Copy link
Copy Markdown
  • Validate provider keys in the disable command to prevent typos or casing issues from reporting a false success.
  • Ensure "disable all" clears all residual metadata (secrets, nonces, and throttles) even when the enabled-providers list is already empty.
  • Add a pre-flight check to backup code generation to avoid storing unusable hashes if the provider cannot be enabled.
  • Update unit tests and documentation to cover these edge cases and state cleanup behaviors.
Open WordPress Playground Preview

- Validate provider keys in the `disable` command to prevent typos or casing issues from reporting a false success.
- Ensure "disable all" clears all residual metadata (secrets, nonces, and throttles) even when the enabled-providers list is already empty.
- Add a pre-flight check to backup code generation to avoid storing unusable hashes if the provider cannot be enabled.
- Update unit tests and documentation to cover these edge cases and state cleanup behaviors.
@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: nimesh-xecurify <nimeshatxecurify@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

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.

1 participant