Skip to content

[PB-6733] remove user migration - #2120

Open
TamaraFinogina wants to merge 2 commits into
masterfrom
remove_user_migration
Open

[PB-6733] remove user migration#2120
TamaraFinogina wants to merge 2 commits into
masterfrom
remove_user_migration

Conversation

@TamaraFinogina

@TamaraFinogina TamaraFinogina commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Description

User migration was a temporary code to allow users with a valid user to remain logged in when we rolled out encrypted users. By now, all users should either have moved to an encrypted user or have had their token expire. Hence, migration is now dead code (all new logins use encrypted users).

Related Issues

Relates to PB-6733

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 login keeps working, as well as downloads

Additional Notes

Make sure no user has a valid session with an unencrypted user. All such sessions should expire before removing the migration. Token lifetime is 3 days and 7 days for refresh.

Encrypted user was merged on 20/08/2026

@TamaraFinogina TamaraFinogina self-assigned this Aug 20, 2026
@TamaraFinogina
TamaraFinogina marked this pull request as ready for review August 28, 2026 13:19
@cloudflare-workers-and-pages

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

Copy link
Copy Markdown

Deploying drive-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: 4dc4dab
Status: ✅  Deploy successful!
Preview URL: https://9da4d89c.drive-web.pages.dev
Branch Preview URL: https://remove-user-migration.drive-web.pages.dev

View logs

@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