Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Microsoft.Kusto: Questions to the Permissions API design #7383

Closed
jrauschenbusch opened this issue Oct 2, 2019 · 1 comment
Closed

Microsoft.Kusto: Questions to the Permissions API design #7383

jrauschenbusch opened this issue Oct 2, 2019 · 1 comment
Assignees
Labels
customer-reported Issues that are reported by GitHub users external to the Azure organization. Service Attention Workflow: This issue is responsible by Azure service team.

Comments

@jrauschenbusch
Copy link

Why there is no dedicated RESTful sub-resource for Kusto Database permissions which uses a unique id like the subscription sub-resource of the service bus API?

Example:
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/servicebus/resource-manager/Microsoft.ServiceBus/stable/2017-04-01/servicebus.json#L2091

Current API:

POST /subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Kusto/clusters/{clusterName}/databases/{databaseName}/addPrincipal

Suggested resource-based API:

PUT /subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Kusto/clusters/{clusterName}/databases/{databaseName}/permissions/{permissionId}

I think the only open point would be to define a meaningful id. But maybe this could be a generated UUID?

And why other properties than role and FQN are required? Imho these two properties would be sufficient to add a permission for a given principal.

@maggiepint maggiepint added customer-reported Issues that are reported by GitHub users external to the Azure organization. Kusto Service Attention Workflow: This issue is responsible by Azure service team. labels Oct 2, 2019
@triage-new-issues triage-new-issues bot removed the triage label Oct 2, 2019
@jrauschenbusch
Copy link
Author

jrauschenbusch commented May 13, 2020

I think this is now solved with #7832.

Dedicated resources for ClusterPrincipalAssignment and DatabasePrincipalAssignment are available since Kusto API 2020-02-15.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
customer-reported Issues that are reported by GitHub users external to the Azure organization. Service Attention Workflow: This issue is responsible by Azure service team.
Projects
None yet
Development

No branches or pull requests

3 participants