Skip to content

Fix: Corrected swapped commands for rotating encryption secrets (Product IS issue #27519)#6051

Merged
hwupathum merged 1 commit intomasterfrom
fixing-product-is-issue-27519-1775742255
Apr 10, 2026
Merged

Fix: Corrected swapped commands for rotating encryption secrets (Product IS issue #27519)#6051
hwupathum merged 1 commit intomasterfrom
fixing-product-is-issue-27519-1775742255

Conversation

@wso2-engineering-bot
Copy link
Copy Markdown

This PR was automatically generated by Claude AI.

  • Fixes The mentioned command is incorrect in rotating encryption secrets for cipher tools product-is#27519
  • Type: Documentation
  • Summary: Corrected the swapped commands for symmetric and asymmetric encryption in the "Rotating Encryption Secrets" section. Symmetric encryption commands now correctly include the -Dsymmetric flag, while asymmetric encryption commands do not.
  • Affected Versions: 6.0.0, 6.1.0, 7.0.0, 7.1.0, 7.2.0, next (fix applied to shared include file)
  • Verification: mkdocs build --strict completed (pre-existing redirect warnings are unrelated to this change)

…uct-is#27519)

The commands for symmetric and asymmetric encryption in the "Rotating Encryption Secrets"
section were incorrectly switched. Symmetric encryption should use the -Dsymmetric flag,
while asymmetric encryption should not.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


wso2-engineering-bot seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 9, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: f7ab91ed-2459-4e2d-b8e2-66573d1a0a9d

📥 Commits

Reviewing files that changed from the base of the PR and between e2f0113 and 001a9c2.

📒 Files selected for processing (1)
  • en/includes/deploy/security/encrypt-passwords-with-cipher-tool.md

📝 Walkthrough

Walkthrough

Updated documentation for cipher tool encryption key rotation commands. Symmetric encryption rotation commands now include the -Dsymmetric flag, while asymmetric encryption commands no longer include it, correcting the previously documented parameter configuration.

Changes

Cohort / File(s) Summary
Documentation Updates
en/includes/deploy/security/encrypt-passwords-with-cipher-tool.md
Corrected ciphertool rotation command parameters in the "Rotating Encryption Secrets" section. Added -Dsymmetric flag to symmetric encryption Linux and Windows commands; removed -Dsymmetric from asymmetric encryption commands.

Suggested labels

Team/Identity Server Core

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The PR description clearly explains the fix and verification steps, but lacks detailed information on test environment and security checks as specified in the template. Add test environment details (JDK versions, OS, browsers tested) and confirm security checks (FindSecurityBugs, no secrets committed) even though this is a documentation-only change.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the primary change: correcting swapped commands for rotating encryption secrets, and includes the issue reference.
Linked Issues check ✅ Passed The PR successfully corrects the swapped cipher-tool commands, applying the -Dsymmetric flag only to symmetric encryption and excluding it from asymmetric encryption as required by issue #27519.
Out of Scope Changes check ✅ Passed All changes are within scope: the PR modifies only the cipher-tool command documentation in the 'Rotating Encryption Secrets' section to correct the swapped symmetric/asymmetric flags.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fixing-product-is-issue-27519-1775742255

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@hwupathum hwupathum merged commit 99589aa into master Apr 10, 2026
3 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The mentioned command is incorrect in rotating encryption secrets for cipher tools

3 participants