Skip to content

LG-10342 Encrypt password digests with multi-region key#8973

Merged
jmhooper merged 4 commits intomainfrom
jmhooper-password-verifier-multi-region
Aug 10, 2023
Merged

LG-10342 Encrypt password digests with multi-region key#8973
jmhooper merged 4 commits intomainfrom
jmhooper-password-verifier-multi-region

Conversation

@jmhooper
Copy link
Contributor

We are working on migrating to a KMS key that supports encryption and decryption across regions. To enable this move we are changing the application to encrypt PII and passwords with both a multi-region and single-region key during the migraiton.

This commit changes the app to encrypt passwords with the single-region key and the multi-region key when the aws_kms_multi_region_write_enabled flag is set.

We are working on migrating to a KMS key that supports encryption and decryption across regions. To enable this move we are changing the application to encrypt PII and passwords with both a multi-region and single-region key during the migraiton.

This commit changes the app to encrypt passwords with the single-region key and the multi-region key when the `aws_kms_multi_region_write_enabled` flag is set.

[skip changelog]
@jmhooper jmhooper requested a review from a team August 10, 2023 15:09
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 00b717e into main Aug 10, 2023
@jmhooper jmhooper deleted the jmhooper-password-verifier-multi-region branch August 10, 2023 18:54
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