-
Notifications
You must be signed in to change notification settings - Fork 300
filename lengths > 255 bytes #937
Safari77
started this conversation in
Ideas & Feature requests
As ZFS 2.3.0 now supports file/dir name lengths of 1023 bytes, how does gocryptfs handle them?
internal/nametransform/longnames.go:ReadLongNameAt() seems to limit them to 255 , but is support for 1023 bytes for gocryptfs possible?
(I am not going to need >255 lengths, but maybe in the future someone will...)
All reactions
Replies: 2 comments
Should be no problem in gocryptfs, looks like the FUSE protocol does support
long names: https://github.com/i-rinat/longnamefs
... On 2025年1月14日, 10:04 Sami Farin, ***@***.***> wrote:
As ZFS 2.3.0 now supports file/dir name lengths of 1023 bytes, how does
gocryptfs handle them?
internal/nametransform/longnames.go:ReadLongNameAt() seems to limit them
to 255 , but is support for 1023 bytes for gocryptfs possible?
(I am not going to need >255 lengths, but maybe in the future someone
will...)
—
Reply to this email directly, view it on GitHub
<#890>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACGA7ZGEBXT3G2KOBFSXVT2KTHLDAVCNFSM6AAAAABVEM2KD6VHI2DSMVQWIX3LMV43ASLTON2WKOZSG44DMNRQHE4DIOA>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
All reactions
0 replies
After changing
gocryptfs/internal/nametransform/names.go
Line 20 in 13773f4
NameMax = 255
to
1023:
$ mkdir cipher plain $ ./gocryptfs -init -plaintextnames cipher $ ./gocryptfs cipher plain $ touch plain/$(python3 -c 'print("a"*100+"b"*100+"c"*100)') $ ls cipher aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaabbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbbcccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccccc gocryptfs.conf $ ls plain ls: reading directory 'plain': Invalid request descriptor
It feels like we're almost there. Hopefully the rest can be implemented fairly easily.
All reactions
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment