qcastel opened a new pull request, #5082:
URL: https://github.com/apache/hadoop/pull/5082

   ### Description of PR
   
   Enforcing public client is a bad idea. We should not weaker our software 
just so the hadoop connector is able to refresh the token.
   Therefore the idea would be to support both public and confidential OAuth2 
client.
   
   The fix is pretty straight forward, it consist of transiting the client 
secret to the azure refresh token grant flow and inject it into the request if 
present.
   
   ### How was this patch tested?
   
   Not yet, will see how I can test it quickly on our side with a patch.
   
   ### For code changes:
   
   - [ x ] Does the title or this PR starts with the corresponding JIRA issue 
id (e.g. 'HADOOP-17799. Your PR title ...')?
   - [ ] Object storage: have the integration tests been executed and the 
endpoint declared according to the connector-specific documentation?
   
   
   


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