diqiu50 opened a new pull request, #13473:
URL: https://github.com/apache/gravitino/pull/13473

   ### What changes were proposed in this pull request?
   
   Documents that `gravitino.client.session.forwardUser` does not support 
`authType=basic`, and explains why.
   
   ### Why are the changes needed?
   
   `authType=basic` requires a real password, but Trino's SPI does not 
propagate the session user's password to connectors after coordinator-side 
authentication, so it cannot support forwardUser like `simple`/`oauth2` do. 
Without this note, users may assume `basic` behaves the same as the other auth 
types and be surprised that every Trino query is authorized as the connector's 
fixed configured user rather than the individual session user.
   
   Fix: #13472
   
   ### Does this PR introduce any user-facing change?
   
   Documentation only — no code change.
   
   ### How was this patch tested?
   
   Docs-only change; no tests required.


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