3bc100aeb3
This adds support for gitignore-like wildcards and exclude patters in reverse mode. It (somewhat) fixes #273: no regexp support, but the syntax should be powerful enough to satisfy most needs. Also, since adding a lot of --exclude options can be tedious, it adds the --exclude-from option to read patterns from a file (or files).
156 lines
7.1 KiB
Go
156 lines
7.1 KiB
Go
package reverse_test
|
|
|
|
import (
|
|
"io/ioutil"
|
|
"path/filepath"
|
|
"testing"
|
|
|
|
"github.com/rfjakob/gocryptfs/internal/ctlsock"
|
|
"github.com/rfjakob/gocryptfs/internal/nametransform"
|
|
"github.com/rfjakob/gocryptfs/tests/test_helpers"
|
|
)
|
|
|
|
const xxx = "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"
|
|
|
|
/*
|
|
tree exclude_test_fs
|
|
exclude_test_fs/
|
|
├── bkp1~
|
|
├── dir1
|
|
│ ├── file1
|
|
│ ├── file2
|
|
│ ├── exclude
|
|
│ ├── longbkp1xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx~
|
|
│ ├── longfile1xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
|
|
│ ├── longfile2xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
|
|
│ ├── longfile3xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
|
|
│ └── subdir1
|
|
│ ├── exclude
|
|
│ └── subdir2
|
|
│ └── exclude
|
|
├── dir2
|
|
│ ├── file
|
|
│ ├── longdir1xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
|
|
│ │ └── file
|
|
│ ├── longfile.xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
|
|
│ └── subdir
|
|
│ └── file
|
|
├── file1
|
|
├── file2
|
|
├── longdir1xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
|
|
│ └── file1
|
|
├── longdir2xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
|
|
│ ├── bkp~
|
|
│ └── file
|
|
├── longfile1xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
|
|
├── longfile2xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
|
|
└── longfile3xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
|
|
*/
|
|
|
|
func ctlsockEncryptPath(t *testing.T, sock string, path string) string {
|
|
req := ctlsock.RequestStruct{EncryptPath: path}
|
|
response := test_helpers.QueryCtlSock(t, sock, req)
|
|
if response.ErrNo != 0 {
|
|
t.Fatal(response)
|
|
}
|
|
return response.Result
|
|
}
|
|
|
|
func testExclude(t *testing.T, flag string) {
|
|
pPatterns := []string{
|
|
"file1", // matches file1 anywhere
|
|
"!longdir1" + xxx + "/file1", // ! includes an otherwise file
|
|
"file2/", // a trailing slash matches only a directory
|
|
"dir1/file2", // matches file2 inside dir1 anywhere
|
|
"#file2", // comments are ignored
|
|
"dir2", // excludes the whole directory
|
|
"longfile2" + xxx, // matches longfile2 anywhere
|
|
"/longfile3" + xxx, // a leading / anchors the match at the root
|
|
"*~", // wildcards are supported
|
|
"dir1/**/exclude", // ** matches any number of directories
|
|
}
|
|
pOk := []string{
|
|
"file2",
|
|
"dir1/longfile1" + xxx,
|
|
"dir1/longfile3" + xxx,
|
|
"longdir1" + xxx,
|
|
"longdir1" + xxx + "/file1",
|
|
"longdir2" + xxx + "/file",
|
|
"longfile1" + xxx,
|
|
}
|
|
pExclude := []string{
|
|
"bkp1~",
|
|
"dir1/file1",
|
|
"dir1/file2",
|
|
"dir1/exclude",
|
|
"dir1/longbkp1" + xxx + "~",
|
|
"dir1/longfile2" + xxx,
|
|
"dir1/subdir1/exclude",
|
|
"dir1/subdir1/subdir2/exclude",
|
|
"dir2",
|
|
"dir2/file",
|
|
"dir2/longdir1" + xxx + "/file",
|
|
"dir2/longfile." + xxx,
|
|
"dir2/subdir",
|
|
"dir2/subdir/file",
|
|
"file1",
|
|
"longdir2" + xxx + "/bkp~",
|
|
"longfile2" + xxx,
|
|
"longfile3" + xxx,
|
|
}
|
|
// Mount reverse fs
|
|
mnt, err := ioutil.TempDir(test_helpers.TmpDir, "TestExclude")
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
sock := mnt + ".sock"
|
|
cliArgs := []string{"-reverse", "-extpass", "echo test", "-ctlsock", sock}
|
|
for _, v := range pPatterns {
|
|
cliArgs = append(cliArgs, flag, v)
|
|
}
|
|
if plaintextnames {
|
|
cliArgs = append(cliArgs, "-config", "exclude_test_fs/.gocryptfs.reverse.conf.plaintextnames")
|
|
}
|
|
test_helpers.MountOrFatal(t, "exclude_test_fs", mnt, cliArgs...)
|
|
defer test_helpers.UnmountPanic(mnt)
|
|
// Get encrypted version of "ok" and "excluded" paths
|
|
cOk := encryptExcludeTestPaths(t, sock, pOk)
|
|
cExclude := encryptExcludeTestPaths(t, sock, pExclude)
|
|
// Check that "excluded" paths are not there and "ok" paths are there
|
|
for _, v := range cExclude {
|
|
if test_helpers.VerifyExistence(mnt + "/" + v) {
|
|
t.Errorf("File %q is visible, but should be excluded", v)
|
|
}
|
|
if nametransform.IsLongContent(filepath.Base(v)) {
|
|
|
|
}
|
|
}
|
|
for _, v := range cOk {
|
|
if !test_helpers.VerifyExistence(mnt + "/" + v) {
|
|
t.Errorf("File %q is hidden, but should be visible", v)
|
|
}
|
|
}
|
|
}
|
|
|
|
// encryptExcludeTestPaths is used by testExclude() to encrypt the lists of
|
|
// testcase paths
|
|
func encryptExcludeTestPaths(t *testing.T, socket string, pRelPaths []string) (out []string) {
|
|
for _, pRelPath := range pRelPaths {
|
|
cRelPath := ctlsockEncryptPath(t, socket, pRelPath)
|
|
out = append(out, cRelPath)
|
|
if !plaintextnames && nametransform.IsLongContent(filepath.Base(cRelPath)) {
|
|
// If we exclude
|
|
// gocryptfs.longname.3vZ_r3eDPb1_fL3j5VA4rd_bcKWLKT9eaxOVIGK5HFA
|
|
// we should also exclude
|
|
// gocryptfs.longname.3vZ_r3eDPb1_fL3j5VA4rd_bcKWLKT9eaxOVIGK5HFA.name
|
|
out = append(out, cRelPath+nametransform.LongNameSuffix)
|
|
}
|
|
}
|
|
return out
|
|
}
|
|
|
|
func TestExclude(t *testing.T) {
|
|
testExclude(t, "-exclude-wildcard")
|
|
testExclude(t, "-ew")
|
|
}
|