Skip to content

Refactoring encrypted recording protos#57055

Merged
eriktate merged 1 commit intomasterfrom
eriktate/refactor-encrypted-recording-protos
Aug 13, 2025
Merged

Refactoring encrypted recording protos#57055
eriktate merged 1 commit intomasterfrom
eriktate/refactor-encrypted-recording-protos

Conversation

@eriktate
Copy link
Copy Markdown
Contributor

This PR adjusts the protos for recording encryption to match some of the recent changes made to the RFD

@eriktate eriktate changed the base branch from master to eriktate/manual-encryption-keys July 22, 2025 20:47
@eriktate eriktate force-pushed the eriktate/manual-encryption-keys branch from 29da268 to 3f8ea79 Compare July 22, 2025 22:15
@eriktate eriktate force-pushed the eriktate/refactor-encrypted-recording-protos branch from 6fe04d5 to 8f04bb8 Compare July 22, 2025 22:19
@eriktate eriktate force-pushed the eriktate/manual-encryption-keys branch from 3f8ea79 to 6c7edeb Compare July 23, 2025 13:02
@eriktate eriktate force-pushed the eriktate/refactor-encrypted-recording-protos branch from 8f04bb8 to 96c32a3 Compare July 23, 2025 13:02
@eriktate eriktate force-pushed the eriktate/manual-encryption-keys branch from 6c7edeb to 174da72 Compare July 23, 2025 13:24
@eriktate eriktate force-pushed the eriktate/refactor-encrypted-recording-protos branch from 96c32a3 to 2dac31e Compare July 23, 2025 13:25
@eriktate eriktate force-pushed the eriktate/manual-encryption-keys branch from 174da72 to 836f806 Compare July 25, 2025 13:21
@eriktate eriktate force-pushed the eriktate/refactor-encrypted-recording-protos branch 2 times, most recently from 5fb1808 to 6c78ec7 Compare July 28, 2025 22:55
@eriktate eriktate marked this pull request as ready for review July 28, 2025 22:57
@github-actions github-actions Bot requested review from hugoShaka and kiosion July 28, 2025 22:57
@eriktate eriktate force-pushed the eriktate/manual-encryption-keys branch from 61d2a6e to 300641f Compare July 29, 2025 16:04
@eriktate eriktate force-pushed the eriktate/refactor-encrypted-recording-protos branch from 6c78ec7 to 3c160d0 Compare July 29, 2025 19:45
@eriktate eriktate added the no-changelog Indicates that a PR does not require a changelog entry label Jul 29, 2025
@eriktate eriktate force-pushed the eriktate/manual-encryption-keys branch 2 times, most recently from 6f668f3 to 27e5b9c Compare July 29, 2025 20:32
@eriktate eriktate force-pushed the eriktate/refactor-encrypted-recording-protos branch from 3c160d0 to e99bcc2 Compare July 29, 2025 21:03
@eriktate eriktate force-pushed the eriktate/manual-encryption-keys branch from d114b67 to 3d01dd2 Compare July 30, 2025 19:48
@eriktate eriktate force-pushed the eriktate/refactor-encrypted-recording-protos branch 2 times, most recently from 7b9deac to 62e79de Compare August 5, 2025 14:24
@eriktate eriktate force-pushed the eriktate/manual-encryption-keys branch from 3d01dd2 to 0372b0a Compare August 5, 2025 20:03
@eriktate eriktate force-pushed the eriktate/refactor-encrypted-recording-protos branch from 62e79de to b222099 Compare August 5, 2025 20:04
@eriktate eriktate force-pushed the eriktate/manual-encryption-keys branch from 0372b0a to 2c98b5d Compare August 5, 2025 20:59
@eriktate eriktate force-pushed the eriktate/refactor-encrypted-recording-protos branch 2 times, most recently from 31d2f5b to ba80699 Compare August 5, 2025 21:22
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Aug 5, 2025

Amplify deployment status

Branch Commit Job ID Status Preview Updated (UTC)
eriktate/refactor-encrypted-recording-protos c82d3de 7 ✅SUCCEED eriktate-refactor-encrypted-recording-protos 2025-08-11 18:25:18

@eriktate eriktate force-pushed the eriktate/refactor-encrypted-recording-protos branch from 93f5e0a to 71ad8e4 Compare August 7, 2025 18:57
@eriktate eriktate force-pushed the eriktate/refactor-encrypted-recording-protos branch from 71ad8e4 to 850c067 Compare August 7, 2025 23:24
@eriktate eriktate force-pushed the eriktate/refactor-encrypted-recording-protos branch from 850c067 to b5a7837 Compare August 8, 2025 20:23
@eriktate
Copy link
Copy Markdown
Contributor Author

@hugoShaka @kiosion friendly bump!

@eriktate eriktate force-pushed the eriktate/refactor-encrypted-recording-protos branch from b5a7837 to c82d3de Compare August 11, 2025 18:17
@rosstimothy rosstimothy requested a review from zmb3 August 12, 2025 13:17
@eriktate eriktate added this pull request to the merge queue Aug 13, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 13, 2025
@eriktate eriktate added this pull request to the merge queue Aug 13, 2025
Merged via the queue into master with commit e298ff3 Aug 13, 2025
44 checks passed
@eriktate eriktate deleted the eriktate/refactor-encrypted-recording-protos branch August 13, 2025 14:06
github-merge-queue Bot pushed a commit that referenced this pull request Sep 3, 2025
* adding support for encryption/decryption keys to keystore manager (#54428, #55652)

* adds new protos for resources related to encrypted session recordings and updates the existing SessionRecordingConfig protos to include a Status (#54780)

* adding local service implementation for recording encryption resources (#54816)

* adding Manager for RecordingEncryption resources that handles shared ops more complex than CRUD (#55078)

* Adding session recording plugin for `age` (#55120)

* adding Manager for RecordingEncryption resources that handles shared ops more complex than CRUD

* adding age plugin wrapping default X25519 Identity/Recipient implementation with hooks to more efficiently lookup private keys given their respective public key

* Adding recording encryption and playback for `sync` modes (#54901)

* adding cache for RecordingEncryption (#55857)

* adding recording_encryption service protos (#55121)

* adding async recording encryption with gRPC multipart uploader (#55859)

* adding file configuration for encrypted session recording (#56200)

* Switching recording encryption to unwrap keys using direct keystore RSA decryption (#56776)

* adding manual key management config (#56920)

* updating protos for recording encryption (#57055)

* Add missing handling for recording encryption configs and keys (#57279)

* updating protos for recording encryption

* changing labels for encryption keys to prevent automatic cleanup, adjusting pkcs11 host UUID check to allow for key sharing of encryption keys, preventing cloud tenants from enabling manual key management, preventing use of recording encryption in FIPS mode

* adding new protos for rotated keys and the local service for interacting (#57576)

with them

* Switching encryption keys from PEM to ASN.1 DER encoding (#58137)

* using pregenerated RSA4096 key for keystore tests because generation is too slow (#58138)

* extending precomputed RSA keys to support 4096-bit keys (#58251)

* adding rotation process to Manager and exposing with new RPCs and (#57577)

* adding rotation sub commands for recording encryption keys and fixing (#57780)

broken session_recording_config when using fileconf

* using more reliable method of validating key bit length
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog Indicates that a PR does not require a changelog entry size/sm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants