Neilk1021 opened a new pull request, #7055:
URL: https://github.com/apache/texera/pull/7055

   ### What changes were proposed in this PR?
   As discussed in #6716 we'd like to split authentication handling into its 
own table `auth_providers.` For future support of multiple sources of 
authentication (e.g. github, IEEE accounts, etc.). This PR migrate the Schema 
from the image on the left to the one on the right.
   
   <img width="247" height="305" alt="image" 
src="https://github.com/user-attachments/assets/6d436b42-8736-4b91-8f1e-d7300b3ad5d2";
 />
   <img width="441" height="292" alt="image" 
src="https://github.com/user-attachments/assets/acffa8e6-4f84-4886-bc94-3627a881523a";
 />
   
   
   #### This PR:
   1. Creates a new `auth_providers` table in texera_ddl.
   2. Providers a migration script `29.sql` to migrate old account data to the 
new schema.
   3. Refactors backend to match new schema.
   4. Refactors portions of `GoogleAuthService` to a generic 
`ExternalAuthProvisioner` to be used in future PRs adding new external logins. 
   
   ### Any related issues, documentation, discussions?
   Closes #7048 
   
   ### How was this PR tested?
   Compiled and ran all tests as well as deployed locally to verify 
functionality. 
   
   ### Was this PR authored or co-authored using generative AI tooling?
   Co-authored with Claude Opus 4.8
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to