Skip to content

Look up emails by previous key fingerprints#6388

Merged
zachmargolis merged 2 commits intomainfrom
margolis-check-previous-fingerprints
May 20, 2022
Merged

Look up emails by previous key fingerprints#6388
zachmargolis merged 2 commits intomainfrom
margolis-check-previous-fingerprints

Conversation

@zachmargolis
Copy link
Contributor

(discovered during our wargames exercise today)

Why: To ensure smoother key rotations

changelog: Bug Fixes, Attribute encryption, Update behavior for rotated keys for email

**Why**: To ensure smoother key rotations

changelog: Bug Fixes, Attribute encryption, Update behavior for rotated keys for email
@zachmargolis zachmargolis requested review from a team, jmhooper and mitchellhenke May 19, 2022 21:13
Copy link
Contributor

@stevegsa stevegsa left a comment

Choose a reason for hiding this comment

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

there's another create_fingerprint in here...

@zachmargolis
Copy link
Contributor Author

there's another create_fingerprint in here...

Good catch! Added in 345f32a, PTAL!

@zachmargolis zachmargolis merged commit 40622d3 into main May 20, 2022
@zachmargolis zachmargolis deleted the margolis-check-previous-fingerprints branch May 20, 2022 18:29
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.

3 participants