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

File shares can't have create permissions #21021

Closed
wants to merge 1 commit into from
Closed

File shares can't have create permissions #21021

wants to merge 1 commit into from

Conversation

rullzer
Copy link
Contributor

@rullzer rullzer commented Dec 8, 2015

Fixes #20839

Since the new sidebar actually is cleaner in how it handles permissions etc we should make sure the permissions are correct.

@rullzer rullzer added this to the 9.0-current milestone Dec 8, 2015
@mention-bot
Copy link

By analyzing the blame information on this pull request, we identified @PVince81 and @icewind1991 to be potential reviewers

@rullzer
Copy link
Contributor Author

rullzer commented Dec 8, 2015

Actually I'm wondering why the permissions are to high to begin with since files don't have create permissions.

@rullzer
Copy link
Contributor Author

rullzer commented Dec 8, 2015

Ok this actually fixes a sympton... we should fix the problem.. => closing

@rullzer rullzer closed this Dec 8, 2015
@rullzer rullzer deleted the fix_20839 branch December 8, 2015 07:39
@lock lock bot locked as resolved and limited conversation to collaborators Aug 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sharing a file allows you to set create permissions
2 participants