Skip to content

Fix Linkis ticket cipher rollback tests - #5491

Open
dajiaohuang wants to merge 2 commits into
apache:masterfrom
dajiaohuang:fix/linkis-test-crypt-key-2327
Open

dajiaohuang wants to merge 2 commits into
apache:masterfrom
dajiaohuang:fix/linkis-test-crypt-key-2327

Conversation

@dajiaohuang

@dajiaohuang dajiaohuang commented Oct 26, 2026 •

Copy link
Copy Markdown

What is the purpose of the change

Addresses two unit-test failures reported in #5482 after applying the captured #5483 fixes. TicketCipher.decrypt recognizes V2 envelopes, but a cipher configured to issue legacy V1 tickets has no AES key, so it throws InvalidKeyException instead of decrypting existing V2 tickets during rollback. Separately, the linkis-module test properties omit the required crypt key, preventing ServerConfigurationTest and SecurityFilterTest from initializing.

Related issues/PRs

Related issue: #5482 (the remaining unit-test failures; this PR does not claim to resolve the other reported CI issues).

Brief change log

  • Keep the V2 issuance choice separate from the AES key used to decrypt V2 envelopes. V1 mode continues issuing DES tickets while retaining the documented ability to decrypt V2 tickets.
  • Add a synthetic, non-deployable wds.linkis.crypt.key only to linkis-module test properties. Production configuration remains unchanged and fail-closed.

Validation

@dajiaohuang dajiaohuang changed the title test(linkis-module): set synthetic crypt key in test resources Fix Linkis ticket cipher rollback tests Oct 26, 2026
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