Skip to content

LG-10342 Encrypt PII with the multi-region encryptor#8958

Merged
jmhooper merged 2 commits intomainfrom
jmhooper-encrypt-mr-pii
Aug 8, 2023
Merged

LG-10342 Encrypt PII with the multi-region encryptor#8958
jmhooper merged 2 commits intomainfrom
jmhooper-encrypt-mr-pii

Conversation

@jmhooper
Copy link
Contributor

@jmhooper jmhooper commented Aug 8, 2023

We are working on migrating to a multi-region capable KMS key. As part of the migration we will be temporarily writing ciphertexts encrypted with the current KMS key and the new multi-region key.

This commit makes a change to the PII encryptor such that it encrypts with the multi-region key when the aws_kms_multi_region_write_enabled flag is set. This will cause a multi-region ciphertext to be returned and written to the database.

We are working on migrating to a multi-region capable KMS key. As part of the migration we will be temporarily writing ciphertexts encrypted with the current KMS key and the new multi-region key.

This commit makes a change to the PII encryptor such that it encrypts with the multi-region key when the `aws_kms_multi_region_write_enabled` flag is set. This will cause a multi-region ciphertext to be returned and written to the database.

[skip changelog]
Copy link
Contributor

@soniaconnolly soniaconnolly left a comment

Choose a reason for hiding this comment

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

LGTM.

@jmhooper jmhooper merged commit acfa1c5 into main Aug 8, 2023
@jmhooper jmhooper deleted the jmhooper-encrypt-mr-pii branch August 8, 2023 18:53
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.

2 participants