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

   <!--
   1. Title: [#<issue>] <type>(<scope>): <subject>
      Examples:
        - "[#123] feat(operator): support xxx"
        - "[#233] fix: check null before access result in xxx"
        - "[MINOR] refactor: fix typo in variable name"
        - "[MINOR] docs: fix typo in README"
        - "[#255] test: fix flaky test NameOfTheTest"
      Reference: https://www.conventionalcommits.org/en/v1.0.0/
   2. If the PR is unfinished, please mark this PR as draft.
   -->
   
   ### What changes were proposed in this pull request?
   
   Adding configs required for Azure based oAuth login
   These will not cause any problems with the current oAuth as these are just 
additional configs exposed in /configs endpoint
   
   ### Why are the changes needed?
   
   These changes will be used in future PRs for Azure MSAL based oAuth support 
in the front end code, through /configs endpoint
   
   Fix: #(issue)
   
   ### Does this PR introduce _any_ user-facing change?
   
   This does not introduce user facing changes. Only changing the /configs API 
response for now
   
   ### How was this patch tested?
   
   (Please test your changes, and provide instructions on how to test it:
   Tested in local deployment.
   Enabled Azure oauth by
   ```
   gravitino.authenticators = oauth
   gravitino.authenticator.oauth.provider = azure
   ```
   and set
   gravitino.authenticator.oauth.azure.client-id , 
gravitino.authenticator.oauth.azure.authority, 
gravitino.authenticator.oauth.azure.redirect-uri, 
gravitino.authenticator.oauth.azure.jwks-uri, 
gravitino.authenticator.oauth.azure.scope
   
   along with existing oauth required parameters (we can modify this in future 
to make it such that if provider is set, the current oauth params need not be 
"required")
   Configs endpoint returns:
   <img width="1226" height="290" alt="Screenshot 2025-07-15 at 10 01 28 AM" 
src="https://github.com/user-attachments/assets/1a078464-fb28-4d05-bb4d-39f86c45b618";
 />
   
   


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