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

Permission Access Control List on unix system #7106

Open
Decodetalkers opened this issue Nov 12, 2022 · 5 comments
Open

Permission Access Control List on unix system #7106

Decodetalkers opened this issue Nov 12, 2022 · 5 comments
Labels
enhancement New feature or request file-system Related to commands and core nushell behavior around the file system platform-specific platform-specific issues

Comments

@Decodetalkers
Copy link
Contributor

Describe the bug

image
As you have seen , except 9 bits of permission, sometimes there are an extra permission , it is called extended permissions, and seems with the permission, every program will have the access to the folder, although they do not have any permission under the folder. and maybe it can be the enhancement of ls command

How to reproduce

  1. plug the U board
  2. Enter /run/media or other place it is mounted
  3. /usr/bin/ls -la or try exa -la . you will see '+' if use ^ls, and @ if use exa

Expected behavior

the xattrs should be handled, related code of exa is in https://github.com/ogham/exa/blob/master/src/fs/fields.rs#L92

Screenshots

No response

Configuration

key value
version 0.71.0
branch makepkg
commit_hash 6cc4ef6
build_os linux-x86_64
build_target x86_64-unknown-linux-gnu
rust_version rustc 1.65.0 (Arch Linux rust 1:1.65.0-1)
cargo_version cargo 1.65.0
pkg_version 0.71.0
build_time 2022-11-09 06:34:03 +00:00
build_rust_channel release
features database, dataframe, default, trash, which, zip
installed_plugins

Additional context

No response

@Decodetalkers
Copy link
Contributor Author

Decodetalkers commented Nov 12, 2022

@Decodetalkers
Copy link
Contributor Author

Maybe ls can also use an extra table to show these information

@Decodetalkers
Copy link
Contributor Author

Decodetalkers commented Nov 12, 2022

But there are not much popular crates about ACLS..and exa also just know the folder is ACLS folder, and people do not always need to enter to the ACLS folder, they just need to enter into the mounted folder of device in it , and in the mounted folders, it will have the known permission. So I think maybe an other solution is to add -i to cd to force change directory 🤔

I don't know if show the message for ACLS is needed,but now exa and origin ls can know the message of such kind of folder. So..

@Decodetalkers Decodetalkers changed the title Permission xattrs on unix system Permission Access Control List on unix system Nov 12, 2022
@sholderbach sholderbach added file-system Related to commands and core nushell behavior around the file system platform-specific platform-specific issues enhancement New feature or request labels Nov 12, 2022
@remmycat
Copy link

remmycat commented Nov 19, 2022

Hey @Decodetalkers 🙂
I ran into the issue of not being able to read xattrs from a different angle (macOS quarantine…)

I prepared a PR for a generic approach to read xattrs in ls, could you maybe have a look at it for feedback? #7158
(Does the PR solve this issue? What would you like to be different?)

@devurandom
Copy link

In addition to ACLs (getfacl), support for arbitrary extended attributes / xattrs (getfattr) would be nice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request file-system Related to commands and core nushell behavior around the file system platform-specific platform-specific issues
Projects
None yet
Development

No branches or pull requests

4 participants