drive-web icon indicating copy to clipboard operation
drive-web copied to clipboard

[PB-4610] Opaque PoC

Open TamaraFinogina opened this issue 3 months ago • 5 comments

Description

This PR is for the PoC for opaque login (just to agree on the API).

During login, we call is2FAorOpaqueNeeded, which tells us if login should be with Opaque or not. The draft implementation for opaque auth is in src/app/auth/services/auth.opaque.ts.

Related Issues

Proof of Concept for PB-4610

Related Pull Requests

Checklist

  • [x] Changes have been tested locally.
  • [x] 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.
  • [x] No new warnings or errors have been introduced.
  • [x] SonarCloud issues have been reviewed and addressed.
  • [ ] QA Passed

Testing Process

No tests are added yet.

Additional Notes

To build, you need to link the SDK from the opaque PoC PR.

TamaraFinogina avatar Oct 14 '25 12:10 TamaraFinogina

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
drive-web Ready Ready Preview Comment Oct 17, 2025 7:29am

vercel[bot] avatar Oct 14 '25 12:10 vercel[bot]

Deploying drive-web with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6a35e91
Status: ✅  Deploy successful!
Preview URL: https://ff7a4d84.drive-web.pages.dev
Branch Preview URL: https://opaque-poc.drive-web.pages.dev

View logs

@larryrider, hash library is not working with node less than 20, so tests on 18 fail due to failed build Screenshot 2025-11-24 at 18 05 09

TamaraFinogina avatar Nov 24 '25 17:11 TamaraFinogina

@larryrider, hash library is not working with node less than 20, so tests on 18 fail due to failed build Screenshot 2025-11-24 at 18 05 09

@TamaraFinogina We should then update those actions to use node 20 or 22

larryrider avatar Nov 24 '25 17:11 larryrider