Project

General

Profile

Bug #28062

Return ACL in filesystem.stat API call

Added by Erin Clark about 1 year ago. Updated 10 months ago.

Status:
Done
Priority:
Expected
Assignee:
Vladimir Vinogradenko
Category:
Middleware
Target version:
Seen in:
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
No
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

As of currently I do not know of a way to get the acl type required for dataset permissions so the component for this is kind of in a half working-state. So I either need the ability to get the acl type for the current directory or have acl not be required so this peice of functionality can safely set permissions properly.


Related issues

Related to FreeNAS - Bug #28083: From Storage-> Volumes -> Edit Permissions in the new UI, the Permission Type only includes Unix and Windows.Done2018-01-302018-03-09
Blocks FreeNAS - Feature #23142: Change Permissions UIDone2017-04-06

Associated revisions

Revision ed588d96 (diff)
Added by Vladimir Vinogradenko about 1 year ago

Return acl in filesystem.stat

Ticket: #28062

History

#1 Updated by Erin Clark about 1 year ago

#2 Updated by Erin Clark about 1 year ago

upon further investigation I will need to have the filesystem type so as to do things like prevent unix permissions changing on windows shares

#3 Updated by Erin Clark about 1 year ago

  • Related to Bug #28083: From Storage-> Volumes -> Edit Permissions in the new UI, the Permission Type only includes Unix and Windows. added

#4 Avatar?id=13649&size=24x24 Updated by Ben Gadd about 1 year ago

  • Due date set to 03/09/2018

#5 Updated by Vladimir Vinogradenko about 1 year ago

  • Status changed from Not Started to Done

filesystem.stat now returns acl

#6 Updated by Dru Lavigne about 1 year ago

  • Subject changed from mp_acl should either be visible in filesystem.stat or not required by storage/permission to Return ACL in filesystem.stat API call
  • Needs Doc changed from Yes to No
  • Needs Merging changed from Yes to No

#7 Updated by Dru Lavigne 12 months ago

  • Status changed from Done to Ready for Testing

#8 Updated by Michael Reynolds 10 months ago

  • Status changed from Ready for Testing to Passed Testing
  • Severity set to New
  • Needs QA changed from Yes to No

Spoke to Vlad about testing/verifying. There is no easy way to test this from an end user perspective as filesystem.stat is an internal call used to distinguish the permission type for a share.

I'm calling this passed since we are able to see permissions for different types of shares and this is the call used to do so.

#9 Updated by Dru Lavigne 10 months ago

  • Status changed from Passed Testing to Done

Also available in: Atom PDF