jtuglu1 commented on PR #19236:
URL: https://github.com/apache/druid/pull/19236#issuecomment-4373451619

   > We could add a PasswordProvider field to IcebergInputSource to support 
that. Then you could use them at runtime in the task to acquire vended 
credentials.
   
   @gianm one other risk here is that this runs the risk of credential 
injection. For example, it does not guarantee that the identity of the user 
POSTing the task is the same as that which is embedded in the field in the 
spec. An authorized user (who can WRITE to that datasource) may be able to 
submit a spec containing the credentials of another user (who has access to 
that input source).


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