mirror of
https://github.com/hanzoai/team.git
synced 2026-09-30 01:57:35 +00:00
zeekay
8feb24ea1f
The server module named github.com/hanzoai/team/server/v8 — a path that resolves to the retired workspace's repository, not this one — while server/public still published github.com/mattermost/mattermost/server/public. Both are github.com/hanzoteam/server/... now, which is where this code actually lives. That second half is what made the binary unbootable. Upstream's public path stayed claimed by upstream, mattermost-plugin-ai imported it, and the build linked public/model twice; public/plugin's gob.Register then ran twice under one name and panicked at init, before main. Reverting the rename traded the brand for a working binary. Rebranding the plugin alongside it needs no such trade: the agents fork is github.com/hanzoteam/agents/v2 against our public module, so no module named mattermost/mattermost is in the graph at all. Measured: `go list -deps ./cmd/mattermost` names exactly one public/model, and it is ours.
149 lines
2.6 KiB
Go
149 lines
2.6 KiB
Go
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
|
|
// See LICENSE.txt for license information.
|
|
|
|
package utils
|
|
|
|
import (
|
|
"archive/zip"
|
|
"errors"
|
|
"os"
|
|
"path/filepath"
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/require"
|
|
|
|
"github.com/hanzoteam/server/server/v8/channels/utils/fileutils"
|
|
)
|
|
|
|
func TestSanitizePath(t *testing.T) {
|
|
cases := []struct {
|
|
input string
|
|
expected string
|
|
}{
|
|
{
|
|
".",
|
|
"",
|
|
},
|
|
{
|
|
"../",
|
|
"",
|
|
},
|
|
{
|
|
"...",
|
|
"",
|
|
},
|
|
{
|
|
"..//.",
|
|
"",
|
|
},
|
|
{
|
|
"/../",
|
|
"",
|
|
},
|
|
{
|
|
"/path/...../to/file",
|
|
"/path/to/file",
|
|
},
|
|
{
|
|
"/path/to/file...",
|
|
"/path/to/file...",
|
|
},
|
|
{
|
|
"/path/to/../../../file",
|
|
"/file",
|
|
},
|
|
{
|
|
"../../../../file",
|
|
"/file",
|
|
},
|
|
{
|
|
"/path/to/file..ext",
|
|
"/path/to/file..ext",
|
|
},
|
|
{
|
|
"/path/to/...file..ext",
|
|
"/path/to/...file..ext",
|
|
},
|
|
{
|
|
"./path/to/...file..ext",
|
|
"path/to/...file..ext",
|
|
},
|
|
{
|
|
"./...file",
|
|
"...file",
|
|
},
|
|
{
|
|
"path/",
|
|
"path",
|
|
},
|
|
}
|
|
|
|
for _, c := range cases {
|
|
t.Run(c.input, func(t *testing.T) {
|
|
require.Equal(t, c.expected, sanitizePath(c.input))
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestUnzipToPath(t *testing.T) {
|
|
testDir, _ := fileutils.FindDir("tests")
|
|
require.NotEmpty(t, testDir)
|
|
|
|
dir, err := os.MkdirTemp("", "unzip")
|
|
require.NoError(t, err)
|
|
defer os.RemoveAll(dir)
|
|
|
|
t.Run("invalid archive", func(t *testing.T) {
|
|
file, err := os.Open(testDir + "/testplugin.tar.gz")
|
|
require.NoError(t, err)
|
|
defer file.Close()
|
|
|
|
info, err := file.Stat()
|
|
require.NoError(t, err)
|
|
|
|
paths, err := UnzipToPath(file, info.Size(), dir)
|
|
require.Error(t, err)
|
|
require.True(t, errors.Is(err, zip.ErrFormat))
|
|
require.Nil(t, paths)
|
|
})
|
|
|
|
t.Run("valid archive", func(t *testing.T) {
|
|
file, err := os.Open(testDir + "/testarchive.zip")
|
|
require.NoError(t, err)
|
|
defer file.Close()
|
|
|
|
info, err := file.Stat()
|
|
require.NoError(t, err)
|
|
|
|
paths, err := UnzipToPath(file, info.Size(), dir)
|
|
require.NoError(t, err)
|
|
require.NotEmpty(t, paths)
|
|
|
|
expectedFiles := map[string]int64{
|
|
dir + "/testfile.txt": 446,
|
|
dir + "/testdir/testfile2.txt": 866,
|
|
dir + "/testdir2/testfile3.txt": 845,
|
|
}
|
|
|
|
expectedDirs := []string{
|
|
dir + "/testdir",
|
|
dir + "/testdir2",
|
|
}
|
|
|
|
err = filepath.Walk(dir, func(path string, info os.FileInfo, err error) error {
|
|
require.NoError(t, err)
|
|
if path == dir {
|
|
return nil
|
|
}
|
|
require.Contains(t, paths, path)
|
|
if info.IsDir() {
|
|
require.Contains(t, expectedDirs, path)
|
|
} else {
|
|
require.Equal(t, expectedFiles[path], info.Size())
|
|
}
|
|
return nil
|
|
})
|
|
require.NoError(t, err)
|
|
})
|
|
}
|