[GitHub] [incubator-druid] capistrant opened a new issue #8826: druid-basic-security - allow LDAP groupMapping updates

[GitBox <gi...@apache.org>]

capistrant opened a new issue #8826: druid-basic-security - allow LDAP groupMapping updates
URL: https://github.com/apache/incubator-druid/issues/8826
 
 
   ### Description
   
   An LDAP group mapping in druid-basic-security can only be created or deleted in its current implementation. This presents a problem when I want to do something like add a role to a mapping. I believe what is needed is an enhancement to allow `PUT` method. This method is idempotent and will create a new group mapping or overwrite an existing group mapping if it already exists. The result always being a group mapping with the attributes listed in the JSON body of the `PUT`.
   
   The role permissions endpoint has similar functionality in which a `POST` request to the permissions endpoint will overwrite any existing permissions list with what is in the JSON body. Instead of using `PUT` for this enhancement, we could mimic that functionality. 
   
   ### Motivation
   
   * Users may want to update groupMappings. They are not going to want to delete and re-create, which is the only mechanism available today according to the docs. Instead they will want to update in one HTTP call.
   

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