Skip to content

[PB-6503] add new hybid encryption for bucket key - #2118

Merged
TamaraFinogina merged 6 commits into
masterfrom
add_new_hybrid
Aug 19, 2026
Merged

[PB-6503] add new hybid encryption for bucket key#2118
TamaraFinogina merged 6 commits into
masterfrom
add_new_hybrid

Conversation

@TamaraFinogina

@TamaraFinogina TamaraFinogina commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Description

This PR adds new hybrid encryption functions + tests. Not used yet. Will be used by a new version of private sharing. Put in a separate PR to reduce the amount of changes.

changes:

  1. Adds new hybrid functions for bucket keys (here we don't need extent, and it's uint8array already, so a bit easier)
  2. Groups same code into subfunctions to reduce duplications
  3. Adds missing checks
  4. Switches to const functionName = () style

Related Pull Requests

Relates to Update private sharings

Checklist

  • Changes have been tested locally.
  • Unit tests have been written or updated as necessary.
  • The code adheres to the repository's coding standards.
  • Relevant documentation has been added or updated.
  • No new warnings or errors have been introduced.
  • SonarCloud issues have been reviewed and addressed.
  • QA Passed

Testing Process

Unit tests

Additional Notes

Nothing should be changed in the existing code; extra parameters are set by default

@TamaraFinogina TamaraFinogina self-assigned this Aug 19, 2026
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 19, 2026

Copy link
Copy Markdown

Deploying drive-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9c3a35b
Status: ✅  Deploy successful!
Preview URL: https://7ef11c60.drive-web.pages.dev
Branch Preview URL: https://add-new-hybrid.drive-web.pages.dev

View logs

@TamaraFinogina
TamaraFinogina marked this pull request as ready for review August 19, 2026 14:56
@TamaraFinogina
TamaraFinogina requested review from a team, CandelR and larryrider as code owners August 19, 2026 14:56
@TamaraFinogina
TamaraFinogina requested a review from xabg2 August 19, 2026 14:58
@sonarqubecloud

Copy link
Copy Markdown

@TamaraFinogina
TamaraFinogina merged commit 9450c87 into master Aug 19, 2026
11 checks passed
@TamaraFinogina
TamaraFinogina deleted the add_new_hybrid branch August 19, 2026 15:23
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