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

Inherited ACL is not recognised as inherited #40

Open
alexkuc opened this issue Dec 11, 2018 · 4 comments
Open

Inherited ACL is not recognised as inherited #40

alexkuc opened this issue Dec 11, 2018 · 4 comments
Assignees
Labels
Question Someone has a question about feature or usability

Comments

@alexkuc
Copy link

alexkuc commented Dec 11, 2018

Hello!

I am running version 4.2.4 of NTFSSecurity on PSv5 (5.1.17763.134). For some reason, NTFSSecurity is not recognising inherited permissions. I have attached screenshots which show this behaviour:

screenshot 2018-12-11 01 17 31

screenshot 2018-12-11 01 18 03

screenshot 2018-12-11 01 18 49

@raandree raandree self-assigned this Dec 11, 2018
@raandree raandree added the Question Someone has a question about feature or usability label Dec 11, 2018
@raandree
Copy link
Owner

Effective access is constructed using the explicit and implicit permissions on a file or folder. The inheritance has an impact on the result but the result itself has no inheritance information. If you calculate the effective access using the Windows Explorer, there is no information about inheritance either.

@alexkuc
Copy link
Author

alexkuc commented Dec 11, 2018

I see. It makes sense. In that case, how come, there is InheritedFrom field? Is it because of the used class for that cmdlet?

@raandree
Copy link
Owner

Right, I am just reusing the same class here to also reuse the same formatter. Is this extremely misleading?

@alexkuc
Copy link
Author

alexkuc commented Dec 11, 2018

I wouldn't say it is extremely misleading. From what I understand in terms of UI, output fields which are empty can cause confusion. I suppose if you could hide InheritedFrom field in this output (as it is not used anyway), it would make it more UI/UX-friendly?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Question Someone has a question about feature or usability
Projects
None yet
Development

No branches or pull requests

2 participants