You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2019/05/10 23:51:08 UTC

[GitHub] [incubator-druid] jon-wei opened a new pull request #7635: Add simpler permissions option to BasicAuthorizer GET APIs

jon-wei opened a new pull request #7635: Add simpler permissions option to BasicAuthorizer GET APIs
URL: https://github.com/apache/incubator-druid/pull/7635
 
 
   This PR adds a `?simplifyPermissions` option to the GET APIs for users and roles in the `druid-basic-security` authorization APIs.
   
   The current output of these APIs can return a list of associated permissions. However, the output format of these permissions has an extra compiled regex field, and it does not match the input format for permissions used by the permissions update APIs.
   
   This inconsistency complicates the use of these APIs for clients such as frontends for editing the authorization configuration, since they must deal with two different permissions formats.
   
   Specifying `?simplifyPermissions` will return consistent output.
   
   This PR deprecates the old formats; in a later release the default output formats should be switched to match the output when `?simplifyPermissions` is used.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org