Skip to content

[PB-6676] Encrypt file and folder tokens - #2080

Open
TamaraFinogina wants to merge 7 commits into
masterfrom
encrypt_other_tokens
Open

[PB-6676] Encrypt file and folder tokens#2080
TamaraFinogina wants to merge 7 commits into
masterfrom
encrypt_other_tokens

Conversation

@TamaraFinogina

@TamaraFinogina TamaraFinogina commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Description

Encrypts file and folder access tokens

Related Issues

Relates to PB-6676

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

Check that actions relying on those tokens are unaffected:

  1. Show folder/file details
  2. Move folder/file to a different location
  3. Show content of the file/folder
  4. Switch between workspaces
  5. Access shared with a member and non-member file/folder

Additional Notes

No need to add this in the hydration function because we don't use it during app render

@TamaraFinogina TamaraFinogina self-assigned this Jul 29, 2026
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 29, 2026

Copy link
Copy Markdown

Deploying drive-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: 09b6e75
Status: ✅  Deploy successful!
Preview URL: https://7ef86837.drive-web.pages.dev
Branch Preview URL: https://encrypt-other-tokens.drive-web.pages.dev

View logs

@TamaraFinogina
TamaraFinogina marked this pull request as ready for review July 29, 2026 13:53
larryrider
larryrider previously approved these changes Jul 30, 2026

@larryrider larryrider left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remember to fix the conflicts

@sonarqubecloud

Copy link
Copy Markdown

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