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

[DLS][Network Drive] Document is visible if Group has GRANT access but user has an explicit DENY #1963

Open
praveen-kukreja opened this issue Dec 8, 2023 · 4 comments · Fixed by #2004
Labels
bug Something isn't working priority:high

Comments

@praveen-kukreja
Copy link
Contributor

praveen-kukreja commented Dec 8, 2023

Bug Description

Steps to Reproduce:

A group A has been provided read/write access to a file in Windows Network Drive but a given user user1 of that group A has an explicit DENY permissions for a document.

Expected Behaviour:

In that case the document is not accessible to user1 in the source (network drive) since in Windows explicit deny ACEs are evaluated before any explicit allowed ACEs. Reference: https://learn.microsoft.com/en-us/windows/win32/secauthz/dacls-and-aces

Actual Results:

The document is visible in the search applications to the user.

Environment

Windows Network Drive

@praveen-kukreja praveen-kukreja added the bug Something isn't working label Dec 8, 2023
@praveen-kukreja praveen-kukreja self-assigned this Dec 8, 2023
@praveen-kukreja
Copy link
Contributor Author

I've found a possible resolution for the fix, implementation is in-progress

@rodmacedo1
Copy link

Hi @praveen-elastic this is the same for #1966?

@praveen-kukreja
Copy link
Contributor Author

Hi @rodmacedo1, yes I'll raise a common PR for both the issues.

@seanstory
Copy link
Member

Based on #2875 (comment), it seems that this issue is not fixed after all. Reopening.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority:high
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants