You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@inlong.apache.org by "gosonzhang (via GitHub)" <gi...@apache.org> on 2023/02/01 03:20:27 UTC

[GitHub] [inlong] gosonzhang opened a new issue, #7300: [Improve][Manager] Replace getRoles() with getAccountType()

gosonzhang opened a new issue, #7300:
URL: https://github.com/apache/inlong/issues/7300

   ### Description
   
   The parent issue: https://github.com/apache/inlong/issues/7168
   
   In the user authorization check of the Open APIs, the getRoles() function of the UserInfo class is used, but the getAccountType() function is used in the current Manager implementation. In order to maintain the consistency of the authentication and authorization logic, the getRoles() calls are replaced with getAccountType()
   
   ### InLong Component
   
   InLong Manager
   
   ### Are you willing to submit PR?
   
   - [X] Yes, I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
   


-- 
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: commits-unsubscribe@inlong.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [inlong] gosonzhang closed issue #7300: [Improve][Manager] Replace getRoles() with getAccountType()

Posted by "gosonzhang (via GitHub)" <gi...@apache.org>.

gosonzhang closed issue #7300: [Improve][Manager] Replace getRoles() with getAccountType()
URL: https://github.com/apache/inlong/issues/7300


-- 
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: commits-unsubscribe@inlong.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [inlong] healchow commented on issue #7300: [Improve][Manager] Replace getRoles() with getAccountType()

Posted by "healchow (via GitHub)" <gi...@apache.org>.

healchow commented on issue #7300:
URL: https://github.com/apache/inlong/issues/7300#issuecomment-1411480624

   How do we manage users of InLong? I think we need to do some research and design, and modify the code after discussion and decision.


-- 
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: commits-unsubscribe@inlong.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org