Skip to content

Linlin1/kek rotation #24816

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

Merged
merged 5 commits into from
Jun 11, 2025
Merged

Linlin1/kek rotation #24816

merged 5 commits into from
Jun 11, 2025

Conversation

Noelin
Copy link
Member

@Noelin Noelin commented Jun 10, 2025

Description

Adding EncryptionKeyVersion for key rotation

@Copilot Copilot AI review requested due to automatic review settings June 10, 2025 20:47
@github-actions github-actions bot added area: server Server related issues (routerlicious) base: main PRs targeted against main branch labels Jun 10, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a new encryption key version to support key rotation.

  • Introduces the "key2026" version in the EncryptionKeyVersion enum.
  • Updates the tenant.ts file to include the new key version.
Comments suppressed due to low confidence (1)

server/routerlicious/packages/services-core/src/tenant.ts:114

  • Please ensure that automated tests are updated to verify support for the new encryption key version 'key2026'.
key2026 = "2026",

@Noelin Noelin merged commit 729b4d3 into microsoft:main Jun 11, 2025
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: server Server related issues (routerlicious) base: main PRs targeted against main branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants