Skip to content

[FR] dont use : in xattr. use . instead #2173

@snoweuph

Description

@snoweuph

Is your feature request related to a problem? Please describe.

I mounted a CIFS as a local disk, that supports cifsacl and xattr.
but due to it beeing cifs, it doesn't allow : inside attributes, as they can't be safely mapped to smb attributes.
thus spaces are currently brocken for me, as they use : inside xattr.

disclaimer: I know that cifs is not a supported feature, but this small change, could increase the compatibility with more filesystems and network mounted filesystems with the POSIX FS driver.

Describe the solution you'd like

don't use : in any attribute keys, instead just use another normal .
like this for example

-user.oc.grant.u:ac4f8e69-cd51-473c-ab90-cc3d2597ca0b = true
+user.oc.grant.u.ac4f8e69-cd51-473c-ab90-cc3d2597ca0b = true

Describe alternatives you've considered

a good alternative would be the option, to store the attributes in normal files, instead of xattr in a sidecar directoy like storage/metadata/xattr.
this would open up the support, for even more filesystems.
generally speaking, i think this would be alot more work, than replacing the usage of : with . for attributes.

Additional context

In cifs : is reserved for NTFS alternate data streams.
smb/cifs generally forbids the free usage of < > : " / \ | ? * in attributes.

Metadata

Metadata

Assignees

No one assigned

    Projects

    Status

    Unqualified

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions