Octavi00 commented on PR #11812: URL: https://github.com/apache/gravitino/pull/11812#issuecomment-4987263134
@lasdf1234, I made your requested change to the IdpUserGroupManager class, and I also wanted to ask if my decision to add a const isBasicLoginRequest inside the axios Index.js file was okay. I know a few days ago you mentioned ignoring the browser popups for a trade off in avoiding header responses on failed logins. Is this a good alternative, or should I also avoid this until I find a better fix / altogether? -- 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]
