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

Better user experience for ADLSGen2 ACLs #3694

Closed
AngshumanNayakMSFT opened this issue Oct 20, 2020 · 1 comment
Closed

Better user experience for ADLSGen2 ACLs #3694

AngshumanNayakMSFT opened this issue Oct 20, 2020 · 1 comment
Assignees
Labels
💡 feature request New feature or request ✅ merged A fix for this issue has been merged
Milestone

Comments

@AngshumanNayakMSFT
Copy link

AngshumanNayakMSFT commented Oct 20, 2020

VSO Item - https://msazure.visualstudio.com/One/_workitems/edit/8495558

Is your feature request related to a problem? Please describe.
Lot of users today open support cases with Storage for ADLSGen2 ACL help as the required R/W ACL permission seems to be not working. In many scenarios the cause is users are not familiar with the POSIX like ACL implementation and that the user needs to have "X" (execute) permission of each of the higher level folder all the way till root.

Describe the solution you'd like
When user selects a R/W access on the "Manage Access" dialog box provide visual cues to the user about this requirement as otherwise the access requests would be denied with 403.

Describe alternatives you've considered
Currently there is no alternative.

Additional context
Add any other context or screenshots about the feature request here.

Suggested screenshot for the UX.

image

@MRayermannMSFT MRayermannMSFT added this to the 1.17.0 milestone Oct 26, 2020
@MRayermannMSFT MRayermannMSFT added the ❔ investigate We need to look into this further label Oct 26, 2020
@MRayermannMSFT MRayermannMSFT self-assigned this Oct 26, 2020
@MRayermannMSFT MRayermannMSFT added the 💡 feature request New feature or request label Nov 2, 2020
@MRayermannMSFT MRayermannMSFT added ✅ merged A fix for this issue has been merged and removed ❔ investigate We need to look into this further labels Dec 8, 2020
@MRayermannMSFT
Copy link
Member

This was merged for inclusion in 1.17

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💡 feature request New feature or request ✅ merged A fix for this issue has been merged
Projects
None yet
Development

No branches or pull requests

2 participants