Project

General

Profile

Bug #30234

Edit Permissions screen needs safety settings for recursive permission changes

Added by Warren Block over 2 years ago. Updated over 2 years ago.

Status:
Done
Priority:
Important
Assignee:
Vaibhav Chauhan
Category:
GUI (new)
Severity:
Reason for Closing:
Reason for Blocked:
Needs QA:
Yes
Needs Doc:
Yes
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

The Edit Permissions option for datasets needs a scary warning and acknowledgement by the user when they enable "Apply permissions recursively". Something like "Changing permissions recursively will affect this directory and all those below it. This can result in data that is inaccessible. Please verify this operation by clicking the "Yes, overwrite permissions for this directory and all directories beneath it" checkbox.

We also should consider whether it should be possible to change permissions or ownership recursively from the root of a volume at all.

Associated revisions

Revision bcf0fc77 (diff)
Added by Erin Clark over 2 years ago

Add warning message to apply permissions recursively Ticket: #30234

History

#1 Updated by Erin Clark over 2 years ago

The root of the volume does not give you the option to set permissions currently, will add the checkbox/warning.

#2 Updated by Erin Clark over 2 years ago

  • Assignee changed from Erin Clark to Vaibhav Chauhan

#3 Updated by Vaibhav Chauhan over 2 years ago

please assign me when PR is ready after build, seems like PR breaks the commit build.

#4 Updated by Vaibhav Chauhan over 2 years ago

  • Assignee changed from Vaibhav Chauhan to Erin Clark

#5 Updated by Dru Lavigne over 2 years ago

  • Status changed from Not Started to In Progress
  • Assignee changed from Erin Clark to Vaibhav Chauhan
  • Target version changed from 11.2-BETA1 to Master - FreeNAS Nightlies

#6 Updated by Dru Lavigne over 2 years ago

  • Status changed from In Progress to Done
  • Needs Merging changed from Yes to No

Also available in: Atom PDF