codeant-ai-for-open-source[bot] commented on PR #38443:
URL: https://github.com/apache/superset/pull/38443#issuecomment-4069401292

   ## **Sequence Diagram**
   
   This PR fixes Google Sheets service account authentication by moving 
credentials into the nested adapter kwargs structure expected by the gsheets 
adapter. It also keeps catalog available for listing while passing it through 
the same adapter path for connection and validation.
   
   ```mermaid
   sequenceDiagram
       participant Admin
       participant GSheetsEngineSpec
       participant SQLAlchemy
       participant GSheetsAdapter
   
       Admin->>GSheetsEngineSpec: Submit encrypted extra with service account 
and catalog
       GSheetsEngineSpec->>GSheetsEngineSpec: Remove oauth2 client info and 
nest service account plus catalog under connect args adapter kwargs gsheetsapi
       GSheetsEngineSpec-->>Admin: Return normalized connection params
   
       Admin->>GSheetsEngineSpec: Run connection validation
       GSheetsEngineSpec->>SQLAlchemy: Create engine using connect args adapter 
kwargs gsheetsapi with credentials and subject
       SQLAlchemy->>GSheetsAdapter: Open connection with nested adapter kwargs
       GSheetsAdapter-->>GSheetsEngineSpec: Validation can proceed with working 
auth
   ```
   
   ---
   *Generated by [CodeAnt AI](https://codeant.ai)*
   


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to