yuqi1129 commented on PR #4160:
URL: https://github.com/apache/gravitino/pull/4160#issuecomment-2252701529

   > > @lw-yang @jerqi The reason front-end tests fail is that as we have 
supported multiple authentications in the server side, front-end page can't get 
right `gravitino.authenticator` and is unable to decide which authentication is 
going to use:
   > > Before modification: gravitino.authenticator=simple 
gravitino.authenticator=oauth
   > > After: 
![image](https://private-user-images.githubusercontent.com/15794564/352055391-0ac38b32-bef5-484d-b026-72ec9b66ab0a.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjE5MTE1OTYsIm5iZiI6MTcyMTkxMTI5NiwicGF0aCI6Ii8xNTc5NDU2NC8zNTIwNTUzOTEtMGFjMzhiMzItYmVmNS00ODRkLWIwMjYtNzJlYzliNjZhYjBhLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MjUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzI1VDEyNDEzNlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTBmZTMzNTc4M2FlOWFlYmMyNTlhYzJkYzhjMzcwODk2NTk5N2I2MTEwMjQ2NTVhZGYxNTEzYWFkNDlhZDgyMmUmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.sHeR8iXDMTtczaon0YUV4q0Zoyi3SjeSdWjCUjKpbyU)
 <img alt="image" width="634" 
src="https://private-user-images.githubusercontent.com/15794564/352055995-a380d7f6-b64d-417e-abeb-a06b97849ca5.png?jwt=eyJhbGc
 
iOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjE5MTE1OTYsIm5iZiI6MTcyMTkxMTI5NiwicGF0aCI6Ii8xNTc5NDU2NC8zNTIwNTU5OTUtYTM4MGQ3ZjYtYjY0ZC00MTdlLWFiZWItYTA2Yjk3ODQ5Y2E1LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MjUlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzI1VDEyNDEzNlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTY3OWM0MjM2ZDFmOWY4OTk1YzA5ZTNmODM1MjVmOTRhY2I4NDc5ZjM1OGNkZjU0YzUyMGY5ZTM0ZWJkMzNjMGYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.z3RL0lh44WQrXRFDHjf5EK5tB6gPeRfMS4hViGLO7Sw">
   > > Note: Please pay attention to the bracket in the value.
   > > Another issue: How will the front-end page know which authentication 
type to use and then load the corresponding login page if the server has set 
multiple authentications?
   > 
   > Front end should use the first authenticator in my opinion.
   
   I concur with it. To make it work, some changes need to be made. 


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