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

Support for Unix ACLs? #928

Open
danmoseley opened this issue Oct 15, 2019 · 5 comments
Open

Support for Unix ACLs? #928

danmoseley opened this issue Oct 15, 2019 · 5 comments
Labels
area-System.IO Cost:L Work that requires one engineer up to 4 weeks Priority:3 Work that is nice to have
Milestone

Comments

@danmoseley
Copy link
Member

What is our story for ACL's on Unix, where file system supports it?

Mono.Posix (what we point at for API for Unix primitives) does not support them. Do we need to offer support for them somewhere?

Whereever that support lies, do we need to add extension methods (like this) to allow objects to be created with the ACL applied? If not, what is the story for folks porting Windows code using ACL's to Linux?

This assumes we do not want to attempt to overload types like DirectorySecurity, which were designed with Windows in mind, to accommodate Unix.

@danmoseley
Copy link
Member Author

The first question here is whether there is any interest, perhaps we can gather that here if any.

@maryamariyan maryamariyan transferred this issue from dotnet/corefx Dec 16, 2019
@Dotnet-GitSync-Bot Dotnet-GitSync-Bot added area-System.IO untriaged New issue has not been triaged by the area owner labels Dec 16, 2019
@JeremyKuhne JeremyKuhne removed the untriaged New issue has not been triaged by the area owner label Feb 26, 2020
@JeremyKuhne JeremyKuhne modified the milestones: 5.0, Future Feb 26, 2020
@JeremyKuhne
Copy link
Member

Triage: Conceptually we want to have ACL support for Linux (see RedHat's support). The reason we haven't done this yet is that there isn't Posix support yet and there hasn't been significant asks from customers yet.

If we do it we'd also want to see if we can provide some sort of common abstraction with the existing Windows ACL support.

@jeffhandley jeffhandley added the Priority:3 Work that is nice to have label Jan 14, 2021
@jeffhandley jeffhandley modified the milestones: Future, 6.0.0 Jan 14, 2021
@carlossanlop carlossanlop added the Cost:L Work that requires one engineer up to 4 weeks label Jan 14, 2021
@jeffhandley jeffhandley modified the milestones: 6.0.0, Future Feb 4, 2021
@jeffhandley
Copy link
Member

I chatted with @carlossanlop on this and we decided to remove it from scope for 6.0. If we want to bring this into scope for a future release, we will need to convert this into a User Story and spend more time understanding the scenarios and identifying some projects that could utilize the feature.

@danmoseley
Copy link
Member Author

Agreed. I think when we are ready to visit this, we should probably step back and think about our story for Unix-specific API's all up, including what's on Mono.Posix.

MichalStrehovsky added a commit to MichalStrehovsky/runtime that referenced this issue Jul 1, 2021
@danmoseley
Copy link
Member Author

@eerhardt based on your recent doc should you close this as out of scope?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-System.IO Cost:L Work that requires one engineer up to 4 weeks Priority:3 Work that is nice to have
Projects
None yet
Development

No branches or pull requests

5 participants