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

services/fs: Proposal: Support file permission and owner #953

Closed
Xuanwo opened this issue Apr 25, 2021 · 0 comments
Closed

services/fs: Proposal: Support file permission and owner #953

Xuanwo opened this issue Apr 25, 2021 · 0 comments

Comments

@Xuanwo
Copy link
Contributor

Xuanwo commented Apr 25, 2021

We could introduce file permission and owner support via add new service pair and metadata.

Proposal

Pair

permission could be set in open, but uid/gid may need a call to Chown.

  • permission: uint32, The permission of this object (for Write related operation)
  • uid: uint32, The uid of this object
  • gid: uint32, The gid of this object

Metadata

We could extract following metadata from f.Sys().(*syscall.Stat_t) (unix only)

  • permission: uint32, The permission of this object
  • uid: uint32, The uid of this object
  • gid: uint32, The gid of this object

Retional

  • How to support permission control on windows? As windows have different concepts compared to Unix alike system.
    • we could just ignore the support for windows for now.
@Xuanwo Xuanwo changed the title Add file permission and owner support Proposal: Support file permission and owner Apr 25, 2021
@Xuanwo Xuanwo changed the title Proposal: Support file permission and owner services/fs: Proposal: Support file permission and owner Oct 22, 2021
@Xuanwo Xuanwo transferred this issue from beyondstorage/go-service-fs Oct 22, 2021
@Xuanwo Xuanwo closed this as completed Sep 15, 2022
@Xuanwo Xuanwo moved this to 📋 Backlog in Xuanwo's Work Sep 15, 2022
@Xuanwo Xuanwo moved this from 📋 Backlog to 📦 Done in Xuanwo's Work Sep 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants