1234567891011121314151617181920212223242526272829303132333435363738394041424344454647484950515253545556575859606162636465666768697071727374757677787980818283848586878889909192939495969798991001011021031041051061071081091101111121131141151161171181191201211221231241251261271281291301311321331341351361371381391401411421431441451461471481491501511521531541551561571581591601611621631641651661671681691701711721731741751761771781791801811821831841851861871881891901911921931941951961971981992002012022032042052062072082092102112122132142152162172182192202212222232242252262272282292302312322332342352362372382392402412422432442452462472482492502512522532542552562572582592602612622632642652662672682692702712722732742752762772782792802812822832842852862872882892902912922932942952962972982993003013023033043053063073083093103113123133143153163173183193203213223233243253263273283293303313323333343353363373383393403413423433443453463473483493503513523533543553563573583593603613623633643653663673683693703713723733743753763773783793803813823833843853863873883893903913923933943953963973983994004014024034044054064074084094104114124134144154164174184194204214224234244254264274284294304314324334344354364374384394404414424434444454464474484494504514524534544554564574584594604614624634644654664674684694704714724734744754764774784794804814824834844854864874884894904914924934944954964974984995005015025035045055065075085095105115125135145155165175185195205215225235245255265275285295305315325335345355365375385395405415425435445455465475485495505515525535545555565575585595605615625635645655665675685695705715725735745755765775785795805815825835845855865875885895905915925935945955965975985996006016026036046056066076086096106116126136146156166176186196206216226236246256266276286296306316326336346356366376386396406416426436446456466476486496506516526536546556566576586596606616626636646656666676686696706716726736746756766776786796806816826836846856866876886896906916926936946956966976986997007017027037047057067077087097107117127137147157167177187197207217227237247257267277287297307317327337347357367377387397407417427437447457467477487497507517527537547557567577587597607617627637647657667677687697707717727737747757767777787797807817827837847857867877887897907917927937947957967977987998008018028038048058068078088098108118128138148158168178188198208218228238248258268278288298308318328338348358368378388398408418428438448458468478488498508518528538548558568578588598608618628638648658668678688698708718728738748758768778788798808818828838848858868878888898908918928938948958968978988999009019029039049059069079089099109119129139149159169179189199209219229239249259269279289299309319329339349359369379389399409419429439449459469479489499509519529539549559569579589599609619629639649659669679689699709719729739749759769779789799809819829839849859869879889899909919929939949959969979989991000100110021003100410051006100710081009101010111012101310141015101610171018101910201021102210231024102510261027102810291030103110321033103410351036103710381039104010411042104310441045104610471048104910501051105210531054105510561057105810591060106110621063106410651066106710681069107010711072107310741075107610771078107910801081108210831084108510861087108810891090109110921093109410951096109710981099110011011102110311041105110611071108110911101111111211131114111511161117111811191120112111221123112411251126112711281129113011311132113311341135113611371138113911401141114211431144114511461147114811491150115111521153115411551156115711581159116011611162116311641165116611671168116911701171117211731174117511761177117811791180118111821183118411851186118711881189119011911192119311941195119611971198119912001201120212031204120512061207120812091210121112121213121412151216121712181219122012211222122312241225122612271228122912301231123212331234123512361237123812391240124112421243124412451246124712481249125012511252125312541255125612571258125912601261126212631264126512661267126812691270127112721273127412751276127712781279128012811282128312841285128612871288128912901291129212931294129512961297129812991300130113021303130413051306130713081309131013111312131313141315131613171318131913201321132213231324132513261327132813291330133113321333133413351336133713381339134013411342134313441345134613471348134913501351135213531354135513561357135813591360136113621363136413651366136713681369137013711372137313741375137613771378137913801381138213831384138513861387138813891390139113921393139413951396139713981399140014011402140314041405140614071408140914101411141214131414141514161417141814191420142114221423142414251426142714281429143014311432143314341435143614371438143914401441144214431444144514461447144814491450145114521453145414551456145714581459146014611462146314641465146614671468146914701471147214731474147514761477147814791480148114821483148414851486148714881489149014911492149314941495149614971498149915001501150215031504150515061507150815091510151115121513151415151516151715181519152015211522152315241525152615271528152915301531153215331534153515361537153815391540154115421543154415451546154715481549155015511552155315541555155615571558155915601561156215631564156515661567156815691570157115721573157415751576157715781579158015811582158315841585158615871588158915901591159215931594159515961597159815991600160116021603160416051606160716081609161016111612161316141615161616171618161916201621162216231624162516261627162816291630163116321633163416351636163716381639164016411642164316441645164616471648164916501651165216531654165516561657165816591660166116621663166416651666166716681669167016711672167316741675167616771678167916801681168216831684168516861687168816891690169116921693169416951696169716981699170017011702170317041705170617071708170917101711171217131714171517161717171817191720172117221723172417251726172717281729173017311732173317341735173617371738173917401741174217431744174517461747174817491750175117521753175417551756175717581759176017611762176317641765176617671768176917701771177217731774177517761777177817791780178117821783178417851786178717881789179017911792179317941795179617971798179918001801180218031804180518061807180818091810181118121813181418151816181718181819182018211822182318241825182618271828182918301831183218331834183518361837183818391840184118421843184418451846184718481849185018511852185318541855185618571858185918601861186218631864186518661867186818691870187118721873187418751876187718781879188018811882188318841885188618871888188918901891189218931894189518961897189818991900190119021903190419051906190719081909191019111912191319141915191619171918191919201921192219231924192519261927192819291930193119321933193419351936193719381939194019411942194319441945194619471948194919501951195219531954195519561957195819591960196119621963196419651966196719681969197019711972197319741975197619771978197919801981198219831984198519861987198819891990199119921993199419951996199719981999200020012002200320042005200620072008200920102011201220132014201520162017201820192020202120222023202420252026202720282029203020312032203320342035203620372038203920402041204220432044204520462047204820492050205120522053205420552056205720582059206020612062206320642065206620672068206920702071207220732074207520762077207820792080208120822083208420852086208720882089209020912092209320942095209620972098209921002101210221032104210521062107210821092110211121122113211421152116211721182119212021212122212321242125212621272128212921302131213221332134213521362137213821392140214121422143214421452146214721482149215021512152215321542155215621572158215921602161216221632164216521662167216821692170217121722173217421752176217721782179218021812182218321842185218621872188218921902191219221932194219521962197219821992200220122022203220422052206220722082209221022112212221322142215221622172218221922202221222222232224222522262227 |
- // Copyright (C) 2019-2022 Nicola Murino
- //
- // This program is free software: you can redistribute it and/or modify
- // it under the terms of the GNU Affero General Public License as published
- // by the Free Software Foundation, version 3.
- //
- // This program is distributed in the hope that it will be useful,
- // but WITHOUT ANY WARRANTY; without even the implied warranty of
- // MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- // GNU Affero General Public License for more details.
- //
- // You should have received a copy of the GNU Affero General Public License
- // along with this program. If not, see <https://www.gnu.org/licenses/>.
- package sftpd
- import (
- "bytes"
- "errors"
- "fmt"
- "io"
- "net"
- "os"
- "path/filepath"
- "runtime"
- "testing"
- "time"
- "github.com/eikenb/pipeat"
- "github.com/pkg/sftp"
- "github.com/rs/xid"
- "github.com/sftpgo/sdk"
- "github.com/stretchr/testify/assert"
- "github.com/stretchr/testify/require"
- "golang.org/x/crypto/ssh"
- "github.com/drakkan/sftpgo/v2/internal/common"
- "github.com/drakkan/sftpgo/v2/internal/dataprovider"
- "github.com/drakkan/sftpgo/v2/internal/kms"
- "github.com/drakkan/sftpgo/v2/internal/util"
- "github.com/drakkan/sftpgo/v2/internal/vfs"
- )
- const (
- osWindows = "windows"
- )
- var (
- configDir = filepath.Join(".", "..", "..")
- )
- type MockChannel struct {
- Buffer *bytes.Buffer
- StdErrBuffer *bytes.Buffer
- ReadError error
- WriteError error
- ShortWriteErr bool
- }
- func (c *MockChannel) Read(data []byte) (int, error) {
- if c.ReadError != nil {
- return 0, c.ReadError
- }
- return c.Buffer.Read(data)
- }
- func (c *MockChannel) Write(data []byte) (int, error) {
- if c.WriteError != nil {
- return 0, c.WriteError
- }
- if c.ShortWriteErr {
- return 0, nil
- }
- return c.Buffer.Write(data)
- }
- func (c *MockChannel) Close() error {
- return nil
- }
- func (c *MockChannel) CloseWrite() error {
- return nil
- }
- func (c *MockChannel) SendRequest(name string, wantReply bool, payload []byte) (bool, error) {
- return true, nil
- }
- func (c *MockChannel) Stderr() io.ReadWriter {
- return c.StdErrBuffer
- }
- // MockOsFs mockable OsFs
- type MockOsFs struct {
- vfs.Fs
- err error
- statErr error
- isAtomicUploadSupported bool
- }
- // Name returns the name for the Fs implementation
- func (fs MockOsFs) Name() string {
- return "mockOsFs"
- }
- // IsUploadResumeSupported returns true if resuming uploads is supported
- func (MockOsFs) IsUploadResumeSupported() bool {
- return false
- }
- // IsAtomicUploadSupported returns true if atomic upload is supported
- func (fs MockOsFs) IsAtomicUploadSupported() bool {
- return fs.isAtomicUploadSupported
- }
- // Stat returns a FileInfo describing the named file
- func (fs MockOsFs) Stat(name string) (os.FileInfo, error) {
- if fs.statErr != nil {
- return nil, fs.statErr
- }
- return os.Stat(name)
- }
- // Lstat returns a FileInfo describing the named file
- func (fs MockOsFs) Lstat(name string) (os.FileInfo, error) {
- if fs.statErr != nil {
- return nil, fs.statErr
- }
- return os.Lstat(name)
- }
- // Remove removes the named file or (empty) directory.
- func (fs MockOsFs) Remove(name string, isDir bool) error {
- if fs.err != nil {
- return fs.err
- }
- return os.Remove(name)
- }
- // Rename renames (moves) source to target
- func (fs MockOsFs) Rename(source, target string) error {
- if fs.err != nil {
- return fs.err
- }
- return os.Rename(source, target)
- }
- func newMockOsFs(err, statErr error, atomicUpload bool, connectionID, rootDir string) vfs.Fs {
- return &MockOsFs{
- Fs: vfs.NewOsFs(connectionID, rootDir, ""),
- err: err,
- statErr: statErr,
- isAtomicUploadSupported: atomicUpload,
- }
- }
- func TestRemoveNonexistentQuotaScan(t *testing.T) {
- assert.False(t, common.QuotaScans.RemoveUserQuotaScan("username"))
- }
- func TestGetOSOpenFlags(t *testing.T) {
- var flags sftp.FileOpenFlags
- flags.Write = true
- flags.Excl = true
- osFlags := getOSOpenFlags(flags)
- assert.NotEqual(t, 0, osFlags&os.O_WRONLY)
- assert.NotEqual(t, 0, osFlags&os.O_EXCL)
- flags.Append = true
- // append flag should be ignored to allow resume
- assert.NotEqual(t, 0, osFlags&os.O_WRONLY)
- assert.NotEqual(t, 0, osFlags&os.O_EXCL)
- }
- func TestUploadResumeInvalidOffset(t *testing.T) {
- testfile := "testfile" //nolint:goconst
- file, err := os.Create(testfile)
- assert.NoError(t, err)
- user := dataprovider.User{
- BaseUser: sdk.BaseUser{
- Username: "testuser",
- },
- }
- fs := vfs.NewOsFs("", os.TempDir(), "")
- conn := common.NewBaseConnection("", common.ProtocolSFTP, "", "", user)
- baseTransfer := common.NewBaseTransfer(file, conn, nil, file.Name(), file.Name(), testfile,
- common.TransferUpload, 10, 0, 0, 0, false, fs, dataprovider.TransferQuota{})
- transfer := newTransfer(baseTransfer, nil, nil, nil)
- _, err = transfer.WriteAt([]byte("test"), 0)
- assert.Error(t, err, "upload with invalid offset must fail")
- if assert.Error(t, transfer.ErrTransfer) {
- assert.EqualError(t, err, transfer.ErrTransfer.Error())
- assert.Contains(t, transfer.ErrTransfer.Error(), "invalid write offset")
- }
- err = transfer.Close()
- if assert.Error(t, err) {
- assert.ErrorIs(t, err, sftp.ErrSSHFxFailure)
- }
- err = os.Remove(testfile)
- assert.NoError(t, err)
- }
- func TestReadWriteErrors(t *testing.T) {
- testfile := "testfile"
- file, err := os.Create(testfile)
- assert.NoError(t, err)
- user := dataprovider.User{
- BaseUser: sdk.BaseUser{
- Username: "testuser",
- },
- }
- fs := vfs.NewOsFs("", os.TempDir(), "")
- conn := common.NewBaseConnection("", common.ProtocolSFTP, "", "", user)
- baseTransfer := common.NewBaseTransfer(file, conn, nil, file.Name(), file.Name(), testfile, common.TransferDownload,
- 0, 0, 0, 0, false, fs, dataprovider.TransferQuota{})
- transfer := newTransfer(baseTransfer, nil, nil, nil)
- err = file.Close()
- assert.NoError(t, err)
- _, err = transfer.WriteAt([]byte("test"), 0)
- assert.Error(t, err, "writing to closed file must fail")
- buf := make([]byte, 32768)
- _, err = transfer.ReadAt(buf, 0)
- assert.Error(t, err, "reading from a closed file must fail")
- err = transfer.Close()
- assert.Error(t, err)
- r, _, err := pipeat.Pipe()
- assert.NoError(t, err)
- baseTransfer = common.NewBaseTransfer(nil, conn, nil, file.Name(), file.Name(), testfile, common.TransferDownload,
- 0, 0, 0, 0, false, fs, dataprovider.TransferQuota{})
- transfer = newTransfer(baseTransfer, nil, r, nil)
- err = transfer.Close()
- assert.NoError(t, err)
- _, err = transfer.ReadAt(buf, 0)
- assert.Error(t, err, "reading from a closed pipe must fail")
- r, w, err := pipeat.Pipe()
- assert.NoError(t, err)
- pipeWriter := vfs.NewPipeWriter(w)
- baseTransfer = common.NewBaseTransfer(nil, conn, nil, file.Name(), file.Name(), testfile, common.TransferDownload,
- 0, 0, 0, 0, false, fs, dataprovider.TransferQuota{})
- transfer = newTransfer(baseTransfer, pipeWriter, nil, nil)
- err = r.Close()
- assert.NoError(t, err)
- errFake := fmt.Errorf("fake upload error")
- go func() {
- time.Sleep(100 * time.Millisecond)
- pipeWriter.Done(errFake)
- }()
- err = transfer.closeIO()
- assert.EqualError(t, err, errFake.Error())
- _, err = transfer.WriteAt([]byte("test"), 0)
- assert.Error(t, err, "writing to closed pipe must fail")
- err = transfer.BaseTransfer.Close()
- assert.EqualError(t, err, errFake.Error())
- err = os.Remove(testfile)
- assert.NoError(t, err)
- assert.Len(t, conn.GetTransfers(), 0)
- }
- func TestUnsupportedListOP(t *testing.T) {
- conn := common.NewBaseConnection("", common.ProtocolSFTP, "", "", dataprovider.User{})
- sftpConn := Connection{
- BaseConnection: conn,
- }
- request := sftp.NewRequest("Unsupported", "")
- _, err := sftpConn.Filelist(request)
- assert.EqualError(t, err, sftp.ErrSSHFxOpUnsupported.Error())
- }
- func TestTransferCancelFn(t *testing.T) {
- testfile := "testfile"
- file, err := os.Create(testfile)
- assert.NoError(t, err)
- isCancelled := false
- cancelFn := func() {
- isCancelled = true
- }
- user := dataprovider.User{
- BaseUser: sdk.BaseUser{
- Username: "testuser",
- },
- }
- fs := vfs.NewOsFs("", os.TempDir(), "")
- conn := common.NewBaseConnection("", common.ProtocolSFTP, "", "", user)
- baseTransfer := common.NewBaseTransfer(file, conn, cancelFn, file.Name(), file.Name(), testfile, common.TransferDownload,
- 0, 0, 0, 0, false, fs, dataprovider.TransferQuota{})
- transfer := newTransfer(baseTransfer, nil, nil, nil)
- errFake := errors.New("fake error, this will trigger cancelFn")
- transfer.TransferError(errFake)
- err = transfer.Close()
- if assert.Error(t, err) {
- assert.ErrorIs(t, err, sftp.ErrSSHFxFailure)
- }
- if assert.Error(t, transfer.ErrTransfer) {
- assert.EqualError(t, transfer.ErrTransfer, errFake.Error())
- }
- assert.True(t, isCancelled, "cancelFn not called!")
- err = os.Remove(testfile)
- assert.NoError(t, err)
- }
- func TestUploadFiles(t *testing.T) {
- common.Config.UploadMode = common.UploadModeAtomic
- fs := vfs.NewOsFs("123", os.TempDir(), "")
- u := dataprovider.User{}
- c := Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSFTP, "", "", u),
- }
- var flags sftp.FileOpenFlags
- flags.Write = true
- flags.Trunc = true
- _, err := c.handleSFTPUploadToExistingFile(fs, flags, "missing_path", "other_missing_path", 0, "/missing_path", nil)
- assert.Error(t, err, "upload to existing file must fail if one or both paths are invalid")
- common.Config.UploadMode = common.UploadModeStandard
- _, err = c.handleSFTPUploadToExistingFile(fs, flags, "missing_path", "other_missing_path", 0, "/missing_path", nil)
- assert.Error(t, err, "upload to existing file must fail if one or both paths are invalid")
- missingFile := "missing/relative/file.txt"
- if runtime.GOOS == osWindows {
- missingFile = "missing\\relative\\file.txt"
- }
- _, err = c.handleSFTPUploadToNewFile(fs, flags, ".", missingFile, "/missing", nil)
- assert.Error(t, err, "upload new file in missing path must fail")
- fs = newMockOsFs(nil, nil, false, "123", os.TempDir())
- f, err := os.CreateTemp("", "temp")
- assert.NoError(t, err)
- err = f.Close()
- assert.NoError(t, err)
- tr, err := c.handleSFTPUploadToExistingFile(fs, flags, f.Name(), f.Name(), 123, f.Name(), nil)
- if assert.NoError(t, err) {
- transfer := tr.(*transfer)
- transfers := c.GetTransfers()
- if assert.Equal(t, 1, len(transfers)) {
- assert.Equal(t, transfers[0].ID, transfer.GetID())
- assert.Equal(t, int64(123), transfer.InitialSize)
- err = transfer.Close()
- assert.NoError(t, err)
- assert.Equal(t, 0, len(c.GetTransfers()))
- }
- }
- err = os.Remove(f.Name())
- assert.NoError(t, err)
- common.Config.UploadMode = common.UploadModeAtomicWithResume
- }
- func TestWithInvalidHome(t *testing.T) {
- u := dataprovider.User{}
- u.HomeDir = "home_rel_path" //nolint:goconst
- _, err := loginUser(&u, dataprovider.LoginMethodPassword, "", nil)
- assert.Error(t, err, "login a user with an invalid home_dir must fail")
- u.HomeDir = os.TempDir()
- fs, err := u.GetFilesystem("123")
- assert.NoError(t, err)
- c := Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSFTP, "", "", u),
- }
- _, err = fs.ResolvePath("../upper_path")
- assert.Error(t, err, "tested path is not a home subdir")
- _, err = c.StatVFS(&sftp.Request{
- Method: "StatVFS",
- Filepath: "../unresolvable-path",
- })
- assert.Error(t, err)
- }
- func TestResolveWithRootDir(t *testing.T) {
- u := dataprovider.User{}
- if runtime.GOOS == osWindows {
- u.HomeDir = "C:\\"
- } else {
- u.HomeDir = "/"
- }
- fs, err := u.GetFilesystem("")
- assert.NoError(t, err)
- rel, err := filepath.Rel(u.HomeDir, os.TempDir())
- assert.NoError(t, err)
- p, err := fs.ResolvePath(rel)
- assert.NoError(t, err, "path %v", p)
- }
- func TestSFTPGetUsedQuota(t *testing.T) {
- u := dataprovider.User{}
- u.HomeDir = "home_rel_path"
- u.Username = "test_invalid_user"
- u.QuotaSize = 4096
- u.QuotaFiles = 1
- u.Permissions = make(map[string][]string)
- u.Permissions["/"] = []string{dataprovider.PermAny}
- connection := Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSFTP, "", "", u),
- }
- quotaResult, _ := connection.HasSpace(false, false, "/")
- assert.False(t, quotaResult.HasSpace)
- }
- func TestSupportedSSHCommands(t *testing.T) {
- cmds := GetSupportedSSHCommands()
- assert.Equal(t, len(supportedSSHCommands), len(cmds))
- for _, c := range cmds {
- assert.True(t, util.Contains(supportedSSHCommands, c))
- }
- }
- func TestSSHCommandPath(t *testing.T) {
- buf := make([]byte, 65535)
- stdErrBuf := make([]byte, 65535)
- mockSSHChannel := MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: nil,
- }
- connection := &Connection{
- channel: &mockSSHChannel,
- BaseConnection: common.NewBaseConnection("", common.ProtocolSSH, "", "", dataprovider.User{}),
- }
- sshCommand := sshCommand{
- command: "test",
- connection: connection,
- args: []string{},
- }
- assert.Equal(t, "", sshCommand.getDestPath())
- sshCommand.args = []string{"-t", "/tmp/../path"}
- assert.Equal(t, "/path", sshCommand.getDestPath())
- sshCommand.args = []string{"-t", "/tmp/"}
- assert.Equal(t, "/tmp/", sshCommand.getDestPath())
- sshCommand.args = []string{"-t", "tmp/"}
- assert.Equal(t, "/tmp/", sshCommand.getDestPath())
- sshCommand.args = []string{"-t", "/tmp/../../../path"}
- assert.Equal(t, "/path", sshCommand.getDestPath())
- sshCommand.args = []string{"-t", ".."}
- assert.Equal(t, "/", sshCommand.getDestPath())
- sshCommand.args = []string{"-t", "."}
- assert.Equal(t, "/", sshCommand.getDestPath())
- sshCommand.args = []string{"-t", "//"}
- assert.Equal(t, "/", sshCommand.getDestPath())
- sshCommand.args = []string{"-t", "../.."}
- assert.Equal(t, "/", sshCommand.getDestPath())
- sshCommand.args = []string{"-t", "/.."}
- assert.Equal(t, "/", sshCommand.getDestPath())
- sshCommand.args = []string{"-f", "/a space.txt"}
- assert.Equal(t, "/a space.txt", sshCommand.getDestPath())
- }
- func TestSSHParseCommandPayload(t *testing.T) {
- cmd := "command -a -f /ab\\ à/some\\ spaces\\ \\ \\(\\).txt"
- name, args, _ := parseCommandPayload(cmd)
- assert.Equal(t, "command", name)
- assert.Equal(t, 3, len(args))
- assert.Equal(t, "/ab à/some spaces ().txt", args[2])
- _, _, err := parseCommandPayload("")
- assert.Error(t, err, "parsing invalid command must fail")
- }
- func TestSSHCommandErrors(t *testing.T) {
- buf := make([]byte, 65535)
- stdErrBuf := make([]byte, 65535)
- readErr := fmt.Errorf("test read error")
- mockSSHChannel := MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: readErr,
- }
- server, client := net.Pipe()
- defer func() {
- err := server.Close()
- assert.NoError(t, err)
- }()
- defer func() {
- err := client.Close()
- assert.NoError(t, err)
- }()
- user := dataprovider.User{}
- user.Permissions = map[string][]string{
- "/": {dataprovider.PermAny},
- }
- connection := Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSSH, "", "", user),
- channel: &mockSSHChannel,
- }
- cmd := sshCommand{
- command: "md5sum",
- connection: &connection,
- args: []string{},
- }
- err := cmd.handle()
- assert.Error(t, err, "ssh command must fail, we are sending a fake error")
- cmd = sshCommand{
- command: "md5sum",
- connection: &connection,
- args: []string{"/../../test_file_ftp.dat"},
- }
- err = cmd.handle()
- assert.Error(t, err, "ssh command must fail, we are requesting an invalid path")
- cmd = sshCommand{
- command: "git-receive-pack",
- connection: &connection,
- args: []string{"/../../testrepo"},
- }
- err = cmd.handle()
- assert.Error(t, err, "ssh command must fail, we are requesting an invalid path")
- user = dataprovider.User{}
- user.Permissions = map[string][]string{
- "/": {dataprovider.PermAny},
- }
- user.HomeDir = filepath.Clean(os.TempDir())
- user.QuotaFiles = 1
- user.UsedQuotaFiles = 2
- cmd.connection.User = user
- _, err = cmd.connection.User.GetFilesystem("123")
- assert.NoError(t, err)
- err = cmd.handle()
- assert.EqualError(t, err, common.ErrQuotaExceeded.Error())
- cmd.connection.User.QuotaFiles = 0
- cmd.connection.User.UsedQuotaFiles = 0
- cmd.connection.User.Permissions = make(map[string][]string)
- cmd.connection.User.Permissions["/"] = []string{dataprovider.PermListItems}
- err = cmd.handle()
- assert.EqualError(t, err, common.ErrPermissionDenied.Error())
- cmd.connection.User.Permissions["/"] = []string{dataprovider.PermAny}
- cmd.command = "invalid_command"
- command, err := cmd.getSystemCommand()
- assert.NoError(t, err)
- err = cmd.executeSystemCommand(command)
- assert.Error(t, err, "invalid command must fail")
- command, err = cmd.getSystemCommand()
- assert.NoError(t, err)
- _, err = command.cmd.StderrPipe()
- assert.NoError(t, err)
- err = cmd.executeSystemCommand(command)
- assert.Error(t, err, "command must fail, pipe was already assigned")
- err = cmd.executeSystemCommand(command)
- assert.Error(t, err, "command must fail, pipe was already assigned")
- command, err = cmd.getSystemCommand()
- assert.NoError(t, err)
- _, err = command.cmd.StdoutPipe()
- assert.NoError(t, err)
- err = cmd.executeSystemCommand(command)
- assert.Error(t, err, "command must fail, pipe was already assigned")
- cmd = sshCommand{
- command: "sftpgo-remove",
- connection: &connection,
- args: []string{"/../../src"},
- }
- err = cmd.handle()
- assert.Error(t, err, "ssh command must fail, we are requesting an invalid path")
- cmd = sshCommand{
- command: "sftpgo-copy",
- connection: &connection,
- args: []string{"/../../test_src", "."},
- }
- err = cmd.handle()
- assert.Error(t, err, "ssh command must fail, we are requesting an invalid path")
- cmd.connection.User.HomeDir = filepath.Clean(os.TempDir())
- cmd = sshCommand{
- command: "sftpgo-copy",
- connection: &connection,
- args: []string{"src", "dst"},
- }
- cmd.connection.User.Permissions = make(map[string][]string)
- cmd.connection.User.Permissions["/"] = []string{dataprovider.PermAny}
- common.WaitForTransfers(1)
- _, err = cmd.getSystemCommand()
- if assert.Error(t, err) {
- assert.Contains(t, err.Error(), common.ErrShuttingDown.Error())
- }
- err = common.Initialize(common.Config, 0)
- assert.NoError(t, err)
- }
- func TestCommandsWithExtensionsFilter(t *testing.T) {
- buf := make([]byte, 65535)
- stdErrBuf := make([]byte, 65535)
- mockSSHChannel := MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- }
- server, client := net.Pipe()
- defer server.Close()
- defer client.Close()
- user := dataprovider.User{
- BaseUser: sdk.BaseUser{
- Username: "test",
- HomeDir: os.TempDir(),
- Status: 1,
- },
- }
- user.Filters.FilePatterns = []sdk.PatternsFilter{
- {
- Path: "/subdir",
- AllowedPatterns: []string{".jpg"},
- DeniedPatterns: []string{},
- },
- }
- connection := &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSSH, "", "", user),
- channel: &mockSSHChannel,
- }
- cmd := sshCommand{
- command: "md5sum",
- connection: connection,
- args: []string{"subdir/test.png"},
- }
- err := cmd.handleHashCommands()
- assert.EqualError(t, err, common.ErrPermissionDenied.Error())
- cmd = sshCommand{
- command: "rsync",
- connection: connection,
- args: []string{"--server", "-vlogDtprze.iLsfxC", ".", "/"},
- }
- _, err = cmd.getSystemCommand()
- assert.EqualError(t, err, errUnsupportedConfig.Error())
- cmd = sshCommand{
- command: "git-receive-pack",
- connection: connection,
- args: []string{"/subdir"},
- }
- _, err = cmd.getSystemCommand()
- assert.EqualError(t, err, errUnsupportedConfig.Error())
- cmd = sshCommand{
- command: "git-receive-pack",
- connection: connection,
- args: []string{"/subdir/dir"},
- }
- _, err = cmd.getSystemCommand()
- assert.EqualError(t, err, errUnsupportedConfig.Error())
- cmd = sshCommand{
- command: "git-receive-pack",
- connection: connection,
- args: []string{"/adir/subdir"},
- }
- _, err = cmd.getSystemCommand()
- assert.NoError(t, err)
- }
- func TestSSHCommandsRemoteFs(t *testing.T) {
- buf := make([]byte, 65535)
- stdErrBuf := make([]byte, 65535)
- mockSSHChannel := MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- }
- user := dataprovider.User{}
- user.FsConfig = vfs.Filesystem{
- Provider: sdk.S3FilesystemProvider,
- S3Config: vfs.S3FsConfig{
- BaseS3FsConfig: sdk.BaseS3FsConfig{
- Bucket: "s3bucket",
- Endpoint: "endpoint",
- Region: "eu-west-1",
- },
- },
- }
- connection := &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSFTP, "", "", user),
- channel: &mockSSHChannel,
- }
- cmd := sshCommand{
- command: "md5sum",
- connection: connection,
- args: []string{},
- }
- command, err := cmd.getSystemCommand()
- assert.NoError(t, err)
- err = cmd.executeSystemCommand(command)
- assert.Error(t, err, "command must fail for a non local filesystem")
- cmd = sshCommand{
- command: "sftpgo-copy",
- connection: connection,
- args: []string{},
- }
- err = cmd.handleSFTPGoCopy()
- assert.Error(t, err)
- cmd = sshCommand{
- command: "sftpgo-remove",
- connection: connection,
- args: []string{},
- }
- err = cmd.handleSFTPGoRemove()
- assert.Error(t, err)
- }
- func TestSSHCmdGetFsErrors(t *testing.T) {
- buf := make([]byte, 65535)
- stdErrBuf := make([]byte, 65535)
- mockSSHChannel := MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- }
- user := dataprovider.User{
- BaseUser: sdk.BaseUser{
- HomeDir: "relative path",
- },
- }
- user.Permissions = map[string][]string{}
- user.Permissions["/"] = []string{dataprovider.PermAny}
- connection := &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSFTP, "", "", user),
- channel: &mockSSHChannel,
- }
- cmd := sshCommand{
- command: "sftpgo-remove",
- connection: connection,
- args: []string{"path"},
- }
- err := cmd.handleSFTPGoRemove()
- assert.Error(t, err)
- cmd = sshCommand{
- command: "sftpgo-copy",
- connection: connection,
- args: []string{"path1", "path2"},
- }
- err = cmd.handleSFTPGoCopy()
- assert.Error(t, err)
- err = os.RemoveAll(user.GetHomeDir())
- assert.NoError(t, err)
- }
- func TestGitVirtualFolders(t *testing.T) {
- permissions := make(map[string][]string)
- permissions["/"] = []string{dataprovider.PermAny}
- user := dataprovider.User{
- BaseUser: sdk.BaseUser{
- Permissions: permissions,
- HomeDir: os.TempDir(),
- },
- }
- conn := &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSFTP, "", "", user),
- }
- cmd := sshCommand{
- command: "git-receive-pack",
- connection: conn,
- args: []string{"/vdir"},
- }
- cmd.connection.User.VirtualFolders = append(cmd.connection.User.VirtualFolders, vfs.VirtualFolder{
- BaseVirtualFolder: vfs.BaseVirtualFolder{
- MappedPath: os.TempDir(),
- },
- VirtualPath: "/vdir",
- })
- _, err := cmd.getSystemCommand()
- assert.NoError(t, err)
- cmd.args = []string{"/"}
- _, err = cmd.getSystemCommand()
- assert.EqualError(t, err, errUnsupportedConfig.Error())
- cmd.args = []string{"/vdir1"}
- _, err = cmd.getSystemCommand()
- assert.NoError(t, err)
- cmd.connection.User.VirtualFolders = nil
- cmd.connection.User.VirtualFolders = append(cmd.connection.User.VirtualFolders, vfs.VirtualFolder{
- BaseVirtualFolder: vfs.BaseVirtualFolder{
- MappedPath: os.TempDir(),
- },
- VirtualPath: "/vdir",
- })
- cmd.args = []string{"/vdir/subdir"}
- _, err = cmd.getSystemCommand()
- assert.NoError(t, err)
- cmd.args = []string{"/adir/subdir"}
- _, err = cmd.getSystemCommand()
- assert.NoError(t, err)
- }
- func TestRsyncOptions(t *testing.T) {
- permissions := make(map[string][]string)
- permissions["/"] = []string{dataprovider.PermAny}
- user := dataprovider.User{
- BaseUser: sdk.BaseUser{
- Permissions: permissions,
- HomeDir: os.TempDir(),
- },
- }
- conn := &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSFTP, "", "", user),
- }
- sshCmd := sshCommand{
- command: "rsync",
- connection: conn,
- args: []string{"--server", "-vlogDtprze.iLsfxC", ".", "/"},
- }
- cmd, err := sshCmd.getSystemCommand()
- assert.NoError(t, err)
- assert.True(t, util.Contains(cmd.cmd.Args, "--safe-links"),
- "--safe-links must be added if the user has the create symlinks permission")
- permissions["/"] = []string{dataprovider.PermDownload, dataprovider.PermUpload, dataprovider.PermCreateDirs,
- dataprovider.PermListItems, dataprovider.PermOverwrite, dataprovider.PermDelete, dataprovider.PermRename}
- user.Permissions = permissions
- conn = &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSFTP, "", "", user),
- }
- sshCmd = sshCommand{
- command: "rsync",
- connection: conn,
- args: []string{"--server", "-vlogDtprze.iLsfxC", ".", "/"},
- }
- cmd, err = sshCmd.getSystemCommand()
- assert.NoError(t, err)
- assert.True(t, util.Contains(cmd.cmd.Args, "--munge-links"),
- "--munge-links must be added if the user has the create symlinks permission")
- sshCmd.connection.User.VirtualFolders = append(sshCmd.connection.User.VirtualFolders, vfs.VirtualFolder{
- BaseVirtualFolder: vfs.BaseVirtualFolder{
- MappedPath: os.TempDir(),
- },
- VirtualPath: "/vdir",
- })
- _, err = sshCmd.getSystemCommand()
- assert.EqualError(t, err, errUnsupportedConfig.Error())
- }
- func TestSystemCommandSizeForPath(t *testing.T) {
- permissions := make(map[string][]string)
- permissions["/"] = []string{dataprovider.PermAny}
- user := dataprovider.User{
- BaseUser: sdk.BaseUser{
- Permissions: permissions,
- HomeDir: os.TempDir(),
- },
- }
- fs, err := user.GetFilesystem("123")
- assert.NoError(t, err)
- conn := &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSFTP, "", "", user),
- }
- sshCmd := sshCommand{
- command: "rsync",
- connection: conn,
- args: []string{"--server", "-vlogDtprze.iLsfxC", ".", "/"},
- }
- _, _, err = sshCmd.getSizeForPath(fs, "missing path")
- assert.NoError(t, err)
- testDir := filepath.Join(os.TempDir(), "dir")
- err = os.MkdirAll(testDir, os.ModePerm)
- assert.NoError(t, err)
- testFile := filepath.Join(testDir, "testfile")
- err = os.WriteFile(testFile, []byte("test content"), os.ModePerm)
- assert.NoError(t, err)
- err = os.Symlink(testFile, testFile+".link")
- assert.NoError(t, err)
- numFiles, size, err := sshCmd.getSizeForPath(fs, testFile+".link")
- assert.NoError(t, err)
- assert.Equal(t, 0, numFiles)
- assert.Equal(t, int64(0), size)
- numFiles, size, err = sshCmd.getSizeForPath(fs, testFile)
- assert.NoError(t, err)
- assert.Equal(t, 1, numFiles)
- assert.Equal(t, int64(12), size)
- if runtime.GOOS != osWindows {
- err = os.Chmod(testDir, 0001)
- assert.NoError(t, err)
- _, _, err = sshCmd.getSizeForPath(fs, testFile)
- assert.Error(t, err)
- err = os.Chmod(testDir, os.ModePerm)
- assert.NoError(t, err)
- }
- err = os.RemoveAll(testDir)
- assert.NoError(t, err)
- }
- func TestSystemCommandErrors(t *testing.T) {
- buf := make([]byte, 65535)
- stdErrBuf := make([]byte, 65535)
- readErr := fmt.Errorf("test read error")
- writeErr := fmt.Errorf("test write error")
- mockSSHChannel := MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: nil,
- WriteError: writeErr,
- }
- permissions := make(map[string][]string)
- permissions["/"] = []string{dataprovider.PermAny}
- homeDir := filepath.Join(os.TempDir(), "adir")
- err := os.MkdirAll(homeDir, os.ModePerm)
- assert.NoError(t, err)
- err = os.WriteFile(filepath.Join(homeDir, "afile"), []byte("content"), os.ModePerm)
- assert.NoError(t, err)
- user := dataprovider.User{
- BaseUser: sdk.BaseUser{
- Permissions: permissions,
- HomeDir: homeDir,
- },
- }
- fs, err := user.GetFilesystem("123")
- assert.NoError(t, err)
- connection := &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSFTP, "", "", user),
- channel: &mockSSHChannel,
- }
- var sshCmd sshCommand
- if runtime.GOOS == osWindows {
- sshCmd = sshCommand{
- command: "dir",
- connection: connection,
- args: []string{"/"},
- }
- } else {
- sshCmd = sshCommand{
- command: "ls",
- connection: connection,
- args: []string{"/"},
- }
- }
- systemCmd, err := sshCmd.getSystemCommand()
- assert.NoError(t, err)
- systemCmd.cmd.Dir = os.TempDir()
- // FIXME: the command completes but the fake client is unable to read the response
- // no error is reported in this case. We can see that the expected code is executed
- // reading the test coverage
- sshCmd.executeSystemCommand(systemCmd) //nolint:errcheck
- mockSSHChannel = MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: readErr,
- WriteError: nil,
- }
- sshCmd.connection.channel = &mockSSHChannel
- baseTransfer := common.NewBaseTransfer(nil, sshCmd.connection.BaseConnection, nil, "", "", "",
- common.TransferUpload, 0, 0, 0, 0, false, fs, dataprovider.TransferQuota{})
- transfer := newTransfer(baseTransfer, nil, nil, nil)
- destBuff := make([]byte, 65535)
- dst := bytes.NewBuffer(destBuff)
- _, err = transfer.copyFromReaderToWriter(dst, sshCmd.connection.channel)
- assert.EqualError(t, err, readErr.Error())
- mockSSHChannel = MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: nil,
- WriteError: nil,
- }
- sshCmd.connection.channel = &mockSSHChannel
- transfer.MaxWriteSize = 1
- _, err = transfer.copyFromReaderToWriter(dst, sshCmd.connection.channel)
- assert.True(t, transfer.Connection.IsQuotaExceededError(err))
- mockSSHChannel = MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: nil,
- WriteError: nil,
- ShortWriteErr: true,
- }
- sshCmd.connection.channel = &mockSSHChannel
- _, err = transfer.copyFromReaderToWriter(sshCmd.connection.channel, dst)
- assert.EqualError(t, err, io.ErrShortWrite.Error())
- transfer.MaxWriteSize = -1
- _, err = transfer.copyFromReaderToWriter(sshCmd.connection.channel, dst)
- assert.True(t, transfer.Connection.IsQuotaExceededError(err))
- baseTransfer = common.NewBaseTransfer(nil, sshCmd.connection.BaseConnection, nil, "", "", "",
- common.TransferDownload, 0, 0, 0, 0, false, fs, dataprovider.TransferQuota{
- AllowedDLSize: 1,
- })
- transfer = newTransfer(baseTransfer, nil, nil, nil)
- mockSSHChannel = MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: nil,
- WriteError: nil,
- }
- sshCmd.connection.channel = &mockSSHChannel
- _, err = transfer.copyFromReaderToWriter(dst, sshCmd.connection.channel)
- if assert.Error(t, err) {
- assert.Contains(t, err.Error(), common.ErrReadQuotaExceeded.Error())
- }
- err = os.RemoveAll(homeDir)
- assert.NoError(t, err)
- }
- func TestCommandGetFsError(t *testing.T) {
- user := dataprovider.User{
- FsConfig: vfs.Filesystem{
- Provider: sdk.CryptedFilesystemProvider,
- },
- }
- conn := &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSFTP, "", "", user),
- }
- sshCmd := sshCommand{
- command: "rsync",
- connection: conn,
- args: []string{"--server", "-vlogDtprze.iLsfxC", ".", "/"},
- }
- _, err := sshCmd.getSystemCommand()
- assert.Error(t, err)
- buf := make([]byte, 65535)
- stdErrBuf := make([]byte, 65535)
- mockSSHChannel := MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: nil,
- }
- conn = &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSCP, "", "", user),
- channel: &mockSSHChannel,
- }
- scpCommand := scpCommand{
- sshCommand: sshCommand{
- command: "scp",
- connection: conn,
- args: []string{"-t", "/tmp"},
- },
- }
- err = scpCommand.handleRecursiveUpload()
- assert.Error(t, err)
- err = scpCommand.handleDownload("")
- assert.Error(t, err)
- }
- func TestGetConnectionInfo(t *testing.T) {
- c := common.ConnectionStatus{
- Username: "test_user",
- ConnectionID: "123",
- ClientVersion: "client",
- RemoteAddress: "127.0.0.1:1234",
- Protocol: common.ProtocolSSH,
- Command: "sha1sum /test_file_ftp.dat",
- }
- info := c.GetConnectionInfo()
- assert.Contains(t, info, "sha1sum /test_file_ftp.dat")
- }
- func TestSCPFileMode(t *testing.T) {
- mode := getFileModeAsString(0, true)
- assert.Equal(t, "0755", mode)
- mode = getFileModeAsString(0700, true)
- assert.Equal(t, "0700", mode)
- mode = getFileModeAsString(0750, true)
- assert.Equal(t, "0750", mode)
- mode = getFileModeAsString(0777, true)
- assert.Equal(t, "0777", mode)
- mode = getFileModeAsString(0640, false)
- assert.Equal(t, "0640", mode)
- mode = getFileModeAsString(0600, false)
- assert.Equal(t, "0600", mode)
- mode = getFileModeAsString(0, false)
- assert.Equal(t, "0644", mode)
- fileMode := uint32(0777)
- fileMode = fileMode | uint32(os.ModeSetgid)
- fileMode = fileMode | uint32(os.ModeSetuid)
- fileMode = fileMode | uint32(os.ModeSticky)
- mode = getFileModeAsString(os.FileMode(fileMode), false)
- assert.Equal(t, "7777", mode)
- fileMode = uint32(0644)
- fileMode = fileMode | uint32(os.ModeSetgid)
- mode = getFileModeAsString(os.FileMode(fileMode), false)
- assert.Equal(t, "4644", mode)
- fileMode = uint32(0600)
- fileMode = fileMode | uint32(os.ModeSetuid)
- mode = getFileModeAsString(os.FileMode(fileMode), false)
- assert.Equal(t, "2600", mode)
- fileMode = uint32(0044)
- fileMode = fileMode | uint32(os.ModeSticky)
- mode = getFileModeAsString(os.FileMode(fileMode), false)
- assert.Equal(t, "1044", mode)
- }
- func TestSCPUploadError(t *testing.T) {
- buf := make([]byte, 65535)
- stdErrBuf := make([]byte, 65535)
- writeErr := fmt.Errorf("test write error")
- mockSSHChannel := MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: nil,
- WriteError: writeErr,
- }
- user := dataprovider.User{
- BaseUser: sdk.BaseUser{
- HomeDir: filepath.Join(os.TempDir()),
- Permissions: make(map[string][]string),
- },
- }
- user.Permissions["/"] = []string{dataprovider.PermAny}
- connection := &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSFTP, "", "", user),
- channel: &mockSSHChannel,
- }
- scpCommand := scpCommand{
- sshCommand: sshCommand{
- command: "scp",
- connection: connection,
- args: []string{"-t", "/"},
- },
- }
- err := scpCommand.handle()
- assert.EqualError(t, err, writeErr.Error())
- mockSSHChannel = MockChannel{
- Buffer: bytes.NewBuffer([]byte("D0755 0 testdir\n")),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: nil,
- WriteError: writeErr,
- }
- err = scpCommand.handleRecursiveUpload()
- assert.EqualError(t, err, writeErr.Error())
- mockSSHChannel = MockChannel{
- Buffer: bytes.NewBuffer([]byte("D0755 a testdir\n")),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: nil,
- WriteError: nil,
- }
- err = scpCommand.handleRecursiveUpload()
- assert.Error(t, err)
- }
- func TestSCPInvalidEndDir(t *testing.T) {
- stdErrBuf := make([]byte, 65535)
- mockSSHChannel := MockChannel{
- Buffer: bytes.NewBuffer([]byte("E\n")),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- }
- connection := &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSFTP, "", "", dataprovider.User{
- BaseUser: sdk.BaseUser{
- HomeDir: os.TempDir(),
- },
- }),
- channel: &mockSSHChannel,
- }
- scpCommand := scpCommand{
- sshCommand: sshCommand{
- command: "scp",
- connection: connection,
- args: []string{"-t", "/tmp"},
- },
- }
- err := scpCommand.handleRecursiveUpload()
- assert.EqualError(t, err, "unacceptable end dir command")
- }
- func TestSCPParseUploadMessage(t *testing.T) {
- buf := make([]byte, 65535)
- stdErrBuf := make([]byte, 65535)
- mockSSHChannel := MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: nil,
- }
- fs := vfs.NewOsFs("", os.TempDir(), "")
- connection := &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSFTP, "", "", dataprovider.User{
- BaseUser: sdk.BaseUser{
- HomeDir: os.TempDir(),
- },
- }),
- channel: &mockSSHChannel,
- }
- scpCommand := scpCommand{
- sshCommand: sshCommand{
- command: "scp",
- connection: connection,
- args: []string{"-t", "/tmp"},
- },
- }
- _, _, err := scpCommand.parseUploadMessage(fs, "invalid")
- assert.Error(t, err, "parsing invalid upload message must fail")
- _, _, err = scpCommand.parseUploadMessage(fs, "D0755 0")
- assert.Error(t, err, "parsing incomplete upload message must fail")
- _, _, err = scpCommand.parseUploadMessage(fs, "D0755 invalidsize testdir")
- assert.Error(t, err, "parsing upload message with invalid size must fail")
- _, _, err = scpCommand.parseUploadMessage(fs, "D0755 0 ")
- assert.Error(t, err, "parsing upload message with invalid name must fail")
- }
- func TestSCPProtocolMessages(t *testing.T) {
- buf := make([]byte, 65535)
- stdErrBuf := make([]byte, 65535)
- readErr := fmt.Errorf("test read error")
- writeErr := fmt.Errorf("test write error")
- mockSSHChannel := MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: readErr,
- WriteError: writeErr,
- }
- connection := &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSCP, "", "", dataprovider.User{}),
- channel: &mockSSHChannel,
- }
- scpCommand := scpCommand{
- sshCommand: sshCommand{
- command: "scp",
- connection: connection,
- args: []string{"-t", "/tmp"},
- },
- }
- _, err := scpCommand.readProtocolMessage()
- assert.EqualError(t, err, readErr.Error())
- err = scpCommand.sendConfirmationMessage()
- assert.EqualError(t, err, writeErr.Error())
- err = scpCommand.sendProtocolMessage("E\n")
- assert.EqualError(t, err, writeErr.Error())
- _, err = scpCommand.getNextUploadProtocolMessage()
- assert.EqualError(t, err, readErr.Error())
- mockSSHChannel = MockChannel{
- Buffer: bytes.NewBuffer([]byte("T1183832947 0 1183833773 0\n")),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: nil,
- WriteError: writeErr,
- }
- scpCommand.connection.channel = &mockSSHChannel
- _, err = scpCommand.getNextUploadProtocolMessage()
- assert.EqualError(t, err, writeErr.Error())
- respBuffer := []byte{0x02}
- protocolErrorMsg := "protocol error msg"
- respBuffer = append(respBuffer, protocolErrorMsg...)
- respBuffer = append(respBuffer, 0x0A)
- mockSSHChannel = MockChannel{
- Buffer: bytes.NewBuffer(respBuffer),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: nil,
- WriteError: nil,
- }
- scpCommand.connection.channel = &mockSSHChannel
- err = scpCommand.readConfirmationMessage()
- if assert.Error(t, err) {
- assert.Equal(t, protocolErrorMsg, err.Error())
- }
- }
- func TestSCPTestDownloadProtocolMessages(t *testing.T) {
- buf := make([]byte, 65535)
- stdErrBuf := make([]byte, 65535)
- readErr := fmt.Errorf("test read error")
- writeErr := fmt.Errorf("test write error")
- mockSSHChannel := MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: readErr,
- WriteError: writeErr,
- }
- connection := &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSCP, "", "", dataprovider.User{}),
- channel: &mockSSHChannel,
- }
- scpCommand := scpCommand{
- sshCommand: sshCommand{
- command: "scp",
- connection: connection,
- args: []string{"-f", "-p", "/tmp"},
- },
- }
- path := "testDir"
- err := os.Mkdir(path, os.ModePerm)
- assert.NoError(t, err)
- stat, err := os.Stat(path)
- assert.NoError(t, err)
- err = scpCommand.sendDownloadProtocolMessages(path, stat)
- assert.EqualError(t, err, writeErr.Error())
- mockSSHChannel = MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: readErr,
- WriteError: nil,
- }
- err = scpCommand.sendDownloadProtocolMessages(path, stat)
- assert.EqualError(t, err, readErr.Error())
- mockSSHChannel = MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: readErr,
- WriteError: writeErr,
- }
- scpCommand.args = []string{"-f", "/tmp"}
- scpCommand.connection.channel = &mockSSHChannel
- err = scpCommand.sendDownloadProtocolMessages(path, stat)
- assert.EqualError(t, err, writeErr.Error())
- mockSSHChannel = MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: readErr,
- WriteError: nil,
- }
- scpCommand.connection.channel = &mockSSHChannel
- err = scpCommand.sendDownloadProtocolMessages(path, stat)
- assert.EqualError(t, err, readErr.Error())
- err = os.Remove(path)
- assert.NoError(t, err)
- }
- func TestSCPCommandHandleErrors(t *testing.T) {
- buf := make([]byte, 65535)
- stdErrBuf := make([]byte, 65535)
- readErr := fmt.Errorf("test read error")
- writeErr := fmt.Errorf("test write error")
- mockSSHChannel := MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: readErr,
- WriteError: writeErr,
- }
- server, client := net.Pipe()
- defer func() {
- err := server.Close()
- assert.NoError(t, err)
- }()
- defer func() {
- err := client.Close()
- assert.NoError(t, err)
- }()
- connection := &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSCP, "", "", dataprovider.User{}),
- channel: &mockSSHChannel,
- }
- scpCommand := scpCommand{
- sshCommand: sshCommand{
- command: "scp",
- connection: connection,
- args: []string{"-f", "/tmp"},
- },
- }
- err := scpCommand.handle()
- assert.EqualError(t, err, readErr.Error())
- scpCommand.args = []string{"-i", "/tmp"}
- err = scpCommand.handle()
- assert.Error(t, err, "invalid scp command must fail")
- }
- func TestSCPErrorsMockFs(t *testing.T) {
- errFake := errors.New("fake error")
- u := dataprovider.User{}
- u.Username = "test"
- u.Permissions = make(map[string][]string)
- u.Permissions["/"] = []string{dataprovider.PermAny}
- u.HomeDir = os.TempDir()
- buf := make([]byte, 65535)
- stdErrBuf := make([]byte, 65535)
- mockSSHChannel := MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- }
- server, client := net.Pipe()
- defer func() {
- err := server.Close()
- assert.NoError(t, err)
- }()
- defer func() {
- err := client.Close()
- assert.NoError(t, err)
- }()
- connection := &Connection{
- channel: &mockSSHChannel,
- BaseConnection: common.NewBaseConnection("", common.ProtocolSCP, "", "", u),
- }
- scpCommand := scpCommand{
- sshCommand: sshCommand{
- command: "scp",
- connection: connection,
- args: []string{"-r", "-t", "/tmp"},
- },
- }
- testfile := filepath.Join(u.HomeDir, "testfile")
- err := os.WriteFile(testfile, []byte("test"), os.ModePerm)
- assert.NoError(t, err)
- fs := newMockOsFs(errFake, nil, true, "123", os.TempDir())
- err = scpCommand.handleUploadFile(fs, testfile, testfile, 0, false, 4, "/testfile")
- assert.NoError(t, err)
- err = os.Remove(testfile)
- assert.NoError(t, err)
- }
- func TestSCPRecursiveDownloadErrors(t *testing.T) {
- buf := make([]byte, 65535)
- stdErrBuf := make([]byte, 65535)
- readErr := fmt.Errorf("test read error")
- writeErr := fmt.Errorf("test write error")
- mockSSHChannel := MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: readErr,
- WriteError: writeErr,
- }
- server, client := net.Pipe()
- defer func() {
- err := server.Close()
- assert.NoError(t, err)
- }()
- defer func() {
- err := client.Close()
- assert.NoError(t, err)
- }()
- fs := vfs.NewOsFs("123", os.TempDir(), "")
- connection := &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSCP, "", "", dataprovider.User{
- BaseUser: sdk.BaseUser{
- HomeDir: os.TempDir(),
- },
- }),
- channel: &mockSSHChannel,
- }
- scpCommand := scpCommand{
- sshCommand: sshCommand{
- command: "scp",
- connection: connection,
- args: []string{"-r", "-f", "/tmp"},
- },
- }
- path := "testDir"
- err := os.Mkdir(path, os.ModePerm)
- assert.NoError(t, err)
- stat, err := os.Stat(path)
- assert.NoError(t, err)
- err = scpCommand.handleRecursiveDownload(fs, "invalid_dir", "invalid_dir", stat)
- assert.EqualError(t, err, writeErr.Error())
- mockSSHChannel = MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: nil,
- WriteError: nil,
- }
- scpCommand.connection.channel = &mockSSHChannel
- err = scpCommand.handleRecursiveDownload(fs, "invalid_dir", "invalid_dir", stat)
- assert.Error(t, err, "recursive upload download must fail for a non existing dir")
- err = os.Remove(path)
- assert.NoError(t, err)
- }
- func TestSCPRecursiveUploadErrors(t *testing.T) {
- buf := make([]byte, 65535)
- stdErrBuf := make([]byte, 65535)
- readErr := fmt.Errorf("test read error")
- writeErr := fmt.Errorf("test write error")
- mockSSHChannel := MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: readErr,
- WriteError: writeErr,
- }
- connection := &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSCP, "", "", dataprovider.User{}),
- channel: &mockSSHChannel,
- }
- scpCommand := scpCommand{
- sshCommand: sshCommand{
- command: "scp",
- connection: connection,
- args: []string{"-r", "-t", "/tmp"},
- },
- }
- err := scpCommand.handleRecursiveUpload()
- assert.Error(t, err, "recursive upload must fail, we send a fake error message")
- mockSSHChannel = MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: readErr,
- WriteError: nil,
- }
- scpCommand.connection.channel = &mockSSHChannel
- err = scpCommand.handleRecursiveUpload()
- assert.Error(t, err, "recursive upload must fail, we send a fake error message")
- }
- func TestSCPCreateDirs(t *testing.T) {
- buf := make([]byte, 65535)
- stdErrBuf := make([]byte, 65535)
- u := dataprovider.User{}
- u.HomeDir = "home_rel_path"
- u.Username = "test"
- u.Permissions = make(map[string][]string)
- u.Permissions["/"] = []string{dataprovider.PermAny}
- mockSSHChannel := MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: nil,
- WriteError: nil,
- }
- fs, err := u.GetFilesystem("123")
- assert.NoError(t, err)
- connection := &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSCP, "", "", u),
- channel: &mockSSHChannel,
- }
- scpCommand := scpCommand{
- sshCommand: sshCommand{
- command: "scp",
- connection: connection,
- args: []string{"-r", "-t", "/tmp"},
- },
- }
- err = scpCommand.handleCreateDir(fs, "invalid_dir")
- assert.Error(t, err, "create invalid dir must fail")
- }
- func TestSCPDownloadFileData(t *testing.T) {
- testfile := "testfile"
- buf := make([]byte, 65535)
- readErr := fmt.Errorf("test read error")
- writeErr := fmt.Errorf("test write error")
- stdErrBuf := make([]byte, 65535)
- mockSSHChannelReadErr := MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: readErr,
- WriteError: nil,
- }
- mockSSHChannelWriteErr := MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: nil,
- WriteError: writeErr,
- }
- fs := vfs.NewOsFs("", os.TempDir(), "")
- connection := &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSCP, "", "", dataprovider.User{BaseUser: sdk.BaseUser{HomeDir: os.TempDir()}}),
- channel: &mockSSHChannelReadErr,
- }
- scpCommand := scpCommand{
- sshCommand: sshCommand{
- command: "scp",
- connection: connection,
- args: []string{"-r", "-f", "/tmp"},
- },
- }
- err := os.WriteFile(testfile, []byte("test"), os.ModePerm)
- assert.NoError(t, err)
- stat, err := os.Stat(testfile)
- assert.NoError(t, err)
- err = scpCommand.sendDownloadFileData(fs, testfile, stat, nil)
- assert.EqualError(t, err, readErr.Error())
- scpCommand.connection.channel = &mockSSHChannelWriteErr
- err = scpCommand.sendDownloadFileData(fs, testfile, stat, nil)
- assert.EqualError(t, err, writeErr.Error())
- scpCommand.args = []string{"-r", "-p", "-f", "/tmp"}
- err = scpCommand.sendDownloadFileData(fs, testfile, stat, nil)
- assert.EqualError(t, err, writeErr.Error())
- scpCommand.connection.channel = &mockSSHChannelReadErr
- err = scpCommand.sendDownloadFileData(fs, testfile, stat, nil)
- assert.EqualError(t, err, readErr.Error())
- err = os.Remove(testfile)
- assert.NoError(t, err)
- }
- func TestSCPUploadFiledata(t *testing.T) {
- testfile := "testfile"
- buf := make([]byte, 65535)
- stdErrBuf := make([]byte, 65535)
- readErr := fmt.Errorf("test read error")
- writeErr := fmt.Errorf("test write error")
- mockSSHChannel := MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: readErr,
- WriteError: writeErr,
- }
- user := dataprovider.User{
- BaseUser: sdk.BaseUser{
- Username: "testuser",
- },
- }
- fs := vfs.NewOsFs("", os.TempDir(), "")
- connection := &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSCP, "", "", user),
- channel: &mockSSHChannel,
- }
- scpCommand := scpCommand{
- sshCommand: sshCommand{
- command: "scp",
- connection: connection,
- args: []string{"-r", "-t", "/tmp"},
- },
- }
- file, err := os.Create(testfile)
- assert.NoError(t, err)
- baseTransfer := common.NewBaseTransfer(file, scpCommand.connection.BaseConnection, nil, file.Name(), file.Name(),
- "/"+testfile, common.TransferDownload, 0, 0, 0, 0, true, fs, dataprovider.TransferQuota{})
- transfer := newTransfer(baseTransfer, nil, nil, nil)
- err = scpCommand.getUploadFileData(2, transfer)
- assert.Error(t, err, "upload must fail, we send a fake write error message")
- mockSSHChannel = MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: readErr,
- WriteError: nil,
- }
- scpCommand.connection.channel = &mockSSHChannel
- file, err = os.Create(testfile)
- assert.NoError(t, err)
- transfer.File = file
- transfer.isFinished = false
- transfer.Connection.AddTransfer(transfer)
- err = scpCommand.getUploadFileData(2, transfer)
- assert.Error(t, err, "upload must fail, we send a fake read error message")
- respBuffer := []byte("12")
- respBuffer = append(respBuffer, 0x02)
- mockSSHChannel = MockChannel{
- Buffer: bytes.NewBuffer(respBuffer),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: nil,
- WriteError: nil,
- }
- scpCommand.connection.channel = &mockSSHChannel
- file, err = os.Create(testfile)
- assert.NoError(t, err)
- baseTransfer.File = file
- transfer = newTransfer(baseTransfer, nil, nil, nil)
- transfer.Connection.AddTransfer(transfer)
- err = scpCommand.getUploadFileData(2, transfer)
- assert.Error(t, err, "upload must fail, we have not enough data to read")
- // the file is already closed so we have an error on trasfer closing
- mockSSHChannel = MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: nil,
- WriteError: nil,
- }
- transfer.Connection.AddTransfer(transfer)
- err = scpCommand.getUploadFileData(0, transfer)
- if assert.Error(t, err) {
- assert.EqualError(t, err, common.ErrTransferClosed.Error())
- }
- mockSSHChannel = MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- ReadError: nil,
- WriteError: nil,
- }
- transfer.Connection.AddTransfer(transfer)
- err = scpCommand.getUploadFileData(2, transfer)
- assert.True(t, errors.Is(err, os.ErrClosed))
- err = os.Remove(testfile)
- assert.NoError(t, err)
- }
- func TestUploadError(t *testing.T) {
- common.Config.UploadMode = common.UploadModeAtomic
- user := dataprovider.User{
- BaseUser: sdk.BaseUser{
- Username: "testuser",
- },
- }
- fs := vfs.NewOsFs("", os.TempDir(), "")
- connection := &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSCP, "", "", user),
- }
- testfile := "testfile"
- fileTempName := "temptestfile"
- file, err := os.Create(fileTempName)
- assert.NoError(t, err)
- baseTransfer := common.NewBaseTransfer(file, connection.BaseConnection, nil, testfile, file.Name(),
- testfile, common.TransferUpload, 0, 0, 0, 0, true, fs, dataprovider.TransferQuota{})
- transfer := newTransfer(baseTransfer, nil, nil, nil)
- errFake := errors.New("fake error")
- transfer.TransferError(errFake)
- err = transfer.Close()
- if assert.Error(t, err) {
- assert.EqualError(t, err, common.ErrGenericFailure.Error())
- }
- if assert.Error(t, transfer.ErrTransfer) {
- assert.EqualError(t, transfer.ErrTransfer, errFake.Error())
- }
- assert.Equal(t, int64(0), transfer.BytesReceived.Load())
- assert.NoFileExists(t, testfile)
- assert.NoFileExists(t, fileTempName)
- common.Config.UploadMode = common.UploadModeAtomicWithResume
- }
- func TestTransferFailingReader(t *testing.T) {
- user := dataprovider.User{
- BaseUser: sdk.BaseUser{
- Username: "testuser",
- HomeDir: os.TempDir(),
- },
- FsConfig: vfs.Filesystem{
- Provider: sdk.CryptedFilesystemProvider,
- CryptConfig: vfs.CryptFsConfig{
- Passphrase: kms.NewPlainSecret("crypt secret"),
- },
- },
- }
- user.Permissions = make(map[string][]string)
- user.Permissions["/"] = []string{dataprovider.PermAny}
- fs := newMockOsFs(nil, nil, true, "", os.TempDir())
- connection := &Connection{
- BaseConnection: common.NewBaseConnection("", common.ProtocolSFTP, "", "", user),
- }
- request := sftp.NewRequest("Open", "afile.txt")
- request.Flags = 27 // read,write,create,truncate
- transfer, err := connection.handleFilewrite(request)
- require.NoError(t, err)
- buf := make([]byte, 32)
- _, err = transfer.ReadAt(buf, 0)
- assert.EqualError(t, err, sftp.ErrSSHFxOpUnsupported.Error())
- if c, ok := transfer.(io.Closer); ok {
- err = c.Close()
- assert.NoError(t, err)
- }
- fsPath := filepath.Join(os.TempDir(), "afile.txt")
- r, _, err := pipeat.Pipe()
- assert.NoError(t, err)
- baseTransfer := common.NewBaseTransfer(nil, connection.BaseConnection, nil, fsPath, fsPath, filepath.Base(fsPath),
- common.TransferUpload, 0, 0, 0, 0, false, fs, dataprovider.TransferQuota{})
- errRead := errors.New("read is not allowed")
- tr := newTransfer(baseTransfer, nil, r, errRead)
- _, err = tr.ReadAt(buf, 0)
- assert.EqualError(t, err, errRead.Error())
- err = tr.Close()
- assert.NoError(t, err)
- tr = newTransfer(baseTransfer, nil, nil, errRead)
- _, err = tr.ReadAt(buf, 0)
- assert.EqualError(t, err, errRead.Error())
- err = tr.Close()
- assert.NoError(t, err)
- err = os.Remove(fsPath)
- assert.NoError(t, err)
- assert.Len(t, connection.GetTransfers(), 0)
- }
- func TestConnectionStatusStruct(t *testing.T) {
- var transfers []common.ConnectionTransfer
- transferUL := common.ConnectionTransfer{
- OperationType: "upload",
- StartTime: util.GetTimeAsMsSinceEpoch(time.Now()),
- Size: 123,
- VirtualPath: "/test.upload",
- }
- transferDL := common.ConnectionTransfer{
- OperationType: "download",
- StartTime: util.GetTimeAsMsSinceEpoch(time.Now()),
- Size: 123,
- VirtualPath: "/test.download",
- }
- transfers = append(transfers, transferUL)
- transfers = append(transfers, transferDL)
- c := common.ConnectionStatus{
- Username: "test",
- ConnectionID: "123",
- ClientVersion: "fakeClient-1.0.0",
- RemoteAddress: "127.0.0.1:1234",
- ConnectionTime: util.GetTimeAsMsSinceEpoch(time.Now()),
- LastActivity: util.GetTimeAsMsSinceEpoch(time.Now()),
- Protocol: "SFTP",
- Transfers: transfers,
- }
- durationString := c.GetConnectionDuration()
- assert.NotEqual(t, 0, len(durationString))
- transfersString := c.GetTransfersAsString()
- assert.NotEqual(t, 0, len(transfersString))
- connInfo := c.GetConnectionInfo()
- assert.NotEqual(t, 0, len(connInfo))
- }
- func TestSupportedSecurityOptions(t *testing.T) {
- c := Configuration{
- KexAlgorithms: supportedKexAlgos,
- MACs: supportedMACs,
- Ciphers: supportedCiphers,
- }
- serverConfig := &ssh.ServerConfig{}
- err := c.configureSecurityOptions(serverConfig)
- assert.NoError(t, err)
- assert.Equal(t, supportedCiphers, serverConfig.Ciphers)
- assert.Equal(t, supportedMACs, serverConfig.MACs)
- assert.Equal(t, supportedKexAlgos, serverConfig.KeyExchanges)
- c.KexAlgorithms = append(c.KexAlgorithms, "not a kex")
- err = c.configureSecurityOptions(serverConfig)
- assert.Error(t, err)
- c.KexAlgorithms = supportedKexAlgos
- c.MACs = []string{
- " hmac-sha2-256-etm@openssh.com ", "hmac-sha2-256",
- " hmac-sha2-512-etm@openssh.com", "hmac-sha2-512 ",
- "hmac-sha1 ", " hmac-sha1-96",
- }
- err = c.configureSecurityOptions(serverConfig)
- assert.NoError(t, err)
- assert.Equal(t, supportedCiphers, serverConfig.Ciphers)
- assert.Equal(t, supportedMACs, serverConfig.MACs)
- assert.Equal(t, supportedKexAlgos, serverConfig.KeyExchanges)
- }
- func TestLoadModuli(t *testing.T) {
- dhGEXSha1 := "diffie-hellman-group-exchange-sha1"
- dhGEXSha256 := "diffie-hellman-group-exchange-sha256"
- c := Configuration{}
- c.Moduli = []string{".", "missing file"}
- err := c.loadModuli(configDir)
- assert.Error(t, err)
- assert.NotContains(t, supportedKexAlgos, dhGEXSha1)
- assert.NotContains(t, supportedKexAlgos, dhGEXSha256)
- assert.Len(t, supportedKexAlgos, 10)
- moduli := []byte("20220414072358 2 6 100 2047 5 F19C2D09AD49978F8A0C1B84168A4011A26F9CD516815934764A319FDC5975FA514AAF11B747D8CA6B3919532BEFB68FA118079473895674F3770F71FBB742F176883841EB3DE679BEF53C6AFE437A662F228B03C1E34B5A0D3909F608CEAA16C1F8131DE11E67878EFD918A89205E5E4DE323054010CA4711F25D466BB7727A016DD3F9F53BDBCE093055A4F2497ADEFB5A2500F9C5C3B0BCD88C6489F4C1CBC7CFB67BA6EABA0195794E4188CE9060F431041AD52FB9BAC4DF7FA536F585FBE67746CD57BFAD67567E9706C24D95C49BE95B759657C6BB5151E2AEA32F4CD557C40298A5C402101520EE8AAB8DFEED6FFC11AAF8036D6345923CFB5D1B922F")
- moduliFile := filepath.Join(os.TempDir(), "moduli")
- err = os.WriteFile(moduliFile, moduli, 0600)
- assert.NoError(t, err)
- c.Moduli = []string{moduliFile}
- err = c.loadModuli(configDir)
- assert.NoError(t, err)
- assert.Contains(t, supportedKexAlgos, dhGEXSha1)
- assert.Contains(t, supportedKexAlgos, dhGEXSha256)
- assert.Len(t, supportedKexAlgos, 12)
- err = os.Remove(moduliFile)
- assert.NoError(t, err)
- }
- func TestLoadHostKeys(t *testing.T) {
- serverConfig := &ssh.ServerConfig{}
- c := Configuration{}
- c.HostKeys = []string{".", "missing file"}
- err := c.checkAndLoadHostKeys(configDir, serverConfig)
- assert.Error(t, err)
- testfile := filepath.Join(os.TempDir(), "invalidkey")
- err = os.WriteFile(testfile, []byte("some bytes"), os.ModePerm)
- assert.NoError(t, err)
- c.HostKeys = []string{testfile}
- err = c.checkAndLoadHostKeys(configDir, serverConfig)
- assert.Error(t, err)
- err = os.Remove(testfile)
- assert.NoError(t, err)
- keysDir := filepath.Join(os.TempDir(), "keys")
- err = os.MkdirAll(keysDir, os.ModePerm)
- assert.NoError(t, err)
- rsaKeyName := filepath.Join(keysDir, defaultPrivateRSAKeyName)
- ecdsaKeyName := filepath.Join(keysDir, defaultPrivateECDSAKeyName)
- ed25519KeyName := filepath.Join(keysDir, defaultPrivateEd25519KeyName)
- nonDefaultKeyName := filepath.Join(keysDir, "akey")
- c.HostKeys = []string{nonDefaultKeyName, rsaKeyName, ecdsaKeyName, ed25519KeyName}
- err = c.checkAndLoadHostKeys(configDir, serverConfig)
- assert.Error(t, err)
- assert.FileExists(t, rsaKeyName)
- assert.FileExists(t, ecdsaKeyName)
- assert.FileExists(t, ed25519KeyName)
- assert.NoFileExists(t, nonDefaultKeyName)
- err = os.Remove(rsaKeyName)
- assert.NoError(t, err)
- err = os.Remove(ecdsaKeyName)
- assert.NoError(t, err)
- err = os.Remove(ed25519KeyName)
- assert.NoError(t, err)
- if runtime.GOOS != osWindows {
- err = os.Chmod(keysDir, 0551)
- assert.NoError(t, err)
- c.HostKeys = nil
- err = c.checkAndLoadHostKeys(keysDir, serverConfig)
- assert.Error(t, err)
- c.HostKeys = []string{rsaKeyName, ecdsaKeyName}
- err = c.checkAndLoadHostKeys(configDir, serverConfig)
- assert.Error(t, err)
- c.HostKeys = []string{ecdsaKeyName, rsaKeyName}
- err = c.checkAndLoadHostKeys(configDir, serverConfig)
- assert.Error(t, err)
- c.HostKeys = []string{ed25519KeyName}
- err = c.checkAndLoadHostKeys(configDir, serverConfig)
- assert.Error(t, err)
- err = os.Chmod(keysDir, 0755)
- assert.NoError(t, err)
- }
- err = os.RemoveAll(keysDir)
- assert.NoError(t, err)
- }
- func TestCertCheckerInitErrors(t *testing.T) {
- c := Configuration{}
- c.TrustedUserCAKeys = []string{".", "missing file"}
- err := c.initializeCertChecker("")
- assert.Error(t, err)
- testfile := filepath.Join(os.TempDir(), "invalidkey")
- err = os.WriteFile(testfile, []byte("some bytes"), os.ModePerm)
- assert.NoError(t, err)
- c.TrustedUserCAKeys = []string{testfile}
- err = c.initializeCertChecker("")
- assert.Error(t, err)
- err = os.Remove(testfile)
- assert.NoError(t, err)
- }
- func TestSFTPSubSystem(t *testing.T) {
- permissions := make(map[string][]string)
- permissions["/"] = []string{dataprovider.PermAny}
- user := &dataprovider.User{
- BaseUser: sdk.BaseUser{
- Permissions: permissions,
- HomeDir: os.TempDir(),
- },
- }
- user.FsConfig.Provider = sdk.AzureBlobFilesystemProvider
- err := ServeSubSystemConnection(user, "connID", nil, nil)
- assert.Error(t, err)
- user.FsConfig.Provider = sdk.LocalFilesystemProvider
- buf := make([]byte, 0, 4096)
- stdErrBuf := make([]byte, 0, 4096)
- mockSSHChannel := &MockChannel{
- Buffer: bytes.NewBuffer(buf),
- StdErrBuffer: bytes.NewBuffer(stdErrBuf),
- }
- // this is 327680 and it will result in packet too long error
- _, err = mockSSHChannel.Write([]byte{0x00, 0x05, 0x00, 0x00, 0x00, 0x00})
- assert.NoError(t, err)
- err = ServeSubSystemConnection(user, "id", mockSSHChannel, mockSSHChannel)
- assert.EqualError(t, err, "packet too long")
- subsystemChannel := newSubsystemChannel(mockSSHChannel, mockSSHChannel)
- n, err := subsystemChannel.Write([]byte{0x00})
- assert.NoError(t, err)
- assert.Equal(t, n, 1)
- err = subsystemChannel.Close()
- assert.NoError(t, err)
- }
- func TestRecoverer(t *testing.T) {
- c := Configuration{}
- c.AcceptInboundConnection(nil, nil)
- connID := "connectionID"
- connection := &Connection{
- BaseConnection: common.NewBaseConnection(connID, common.ProtocolSFTP, "", "", dataprovider.User{}),
- }
- c.handleSftpConnection(nil, connection)
- sshCmd := sshCommand{
- command: "cd",
- connection: connection,
- }
- err := sshCmd.handle()
- assert.EqualError(t, err, common.ErrGenericFailure.Error())
- scpCmd := scpCommand{
- sshCommand: sshCommand{
- command: "scp",
- connection: connection,
- },
- }
- err = scpCmd.handle()
- assert.EqualError(t, err, common.ErrGenericFailure.Error())
- assert.Len(t, common.Connections.GetStats(""), 0)
- }
- func TestListernerAcceptErrors(t *testing.T) {
- errFake := errors.New("a fake error")
- listener := newFakeListener(errFake)
- c := Configuration{}
- err := c.serve(listener, nil)
- require.EqualError(t, err, errFake.Error())
- err = listener.Close()
- require.NoError(t, err)
- errNetFake := &fakeNetError{error: errFake}
- listener = newFakeListener(errNetFake)
- err = c.serve(listener, nil)
- require.EqualError(t, err, errFake.Error())
- err = listener.Close()
- require.NoError(t, err)
- }
- type fakeNetError struct {
- error
- count int
- }
- func (e *fakeNetError) Timeout() bool {
- return false
- }
- func (e *fakeNetError) Temporary() bool {
- e.count++
- return e.count < 10
- }
- func (e *fakeNetError) Error() string {
- return e.error.Error()
- }
- type fakeListener struct {
- server net.Conn
- client net.Conn
- err error
- }
- func (l *fakeListener) Accept() (net.Conn, error) {
- return l.client, l.err
- }
- func (l *fakeListener) Close() error {
- errClient := l.client.Close()
- errServer := l.server.Close()
- if errServer != nil {
- return errServer
- }
- return errClient
- }
- func (l *fakeListener) Addr() net.Addr {
- return l.server.LocalAddr()
- }
- func newFakeListener(err error) net.Listener {
- server, client := net.Pipe()
- return &fakeListener{
- server: server,
- client: client,
- err: err,
- }
- }
- func TestFolderPrefix(t *testing.T) {
- c := Configuration{
- FolderPrefix: "files",
- }
- c.checkFolderPrefix()
- assert.Equal(t, "/files", c.FolderPrefix)
- c.FolderPrefix = ""
- c.checkFolderPrefix()
- assert.Empty(t, c.FolderPrefix)
- c.FolderPrefix = "/"
- c.checkFolderPrefix()
- assert.Empty(t, c.FolderPrefix)
- c.FolderPrefix = "/."
- c.checkFolderPrefix()
- assert.Empty(t, c.FolderPrefix)
- c.FolderPrefix = "."
- c.checkFolderPrefix()
- assert.Empty(t, c.FolderPrefix)
- c.FolderPrefix = ".."
- c.checkFolderPrefix()
- assert.Empty(t, c.FolderPrefix)
- }
- func TestLoadRevokedUserCertsFile(t *testing.T) {
- r := revokedCertificates{
- certs: map[string]bool{},
- }
- err := r.load()
- assert.NoError(t, err)
- r.filePath = filepath.Join(os.TempDir(), "sub", "testrevoked")
- err = os.MkdirAll(filepath.Dir(r.filePath), os.ModePerm)
- assert.NoError(t, err)
- err = os.WriteFile(r.filePath, []byte(`no json`), 0644)
- assert.NoError(t, err)
- err = r.load()
- assert.Error(t, err)
- r.filePath = filepath.Dir(r.filePath)
- err = r.load()
- assert.Error(t, err)
- err = os.RemoveAll(r.filePath)
- assert.NoError(t, err)
- }
- func TestMaxUserSessions(t *testing.T) {
- connection := &Connection{
- BaseConnection: common.NewBaseConnection(xid.New().String(), common.ProtocolSFTP, "", "", dataprovider.User{
- BaseUser: sdk.BaseUser{
- Username: "user_max_sessions",
- HomeDir: filepath.Clean(os.TempDir()),
- MaxSessions: 1,
- },
- }),
- }
- err := common.Connections.Add(connection)
- assert.NoError(t, err)
- c := Configuration{}
- c.handleSftpConnection(nil, connection)
- sshCmd := sshCommand{
- command: "cd",
- connection: connection,
- }
- err = sshCmd.handle()
- if assert.Error(t, err) {
- assert.Contains(t, err.Error(), "too many open sessions")
- }
- scpCmd := scpCommand{
- sshCommand: sshCommand{
- command: "scp",
- connection: connection,
- },
- }
- err = scpCmd.handle()
- if assert.Error(t, err) {
- assert.Contains(t, err.Error(), "too many open sessions")
- }
- err = ServeSubSystemConnection(&connection.User, connection.ID, nil, nil)
- if assert.Error(t, err) {
- assert.Contains(t, err.Error(), "too many open sessions")
- }
- common.Connections.Remove(connection.GetID())
- assert.Len(t, common.Connections.GetStats(""), 0)
- }
- func TestCanReadSymlink(t *testing.T) {
- connection := &Connection{
- BaseConnection: common.NewBaseConnection(xid.New().String(), common.ProtocolSFTP, "", "", dataprovider.User{
- BaseUser: sdk.BaseUser{
- Username: "user_can_read_symlink",
- HomeDir: filepath.Clean(os.TempDir()),
- Permissions: map[string][]string{
- "/": {dataprovider.PermAny},
- "/sub": {dataprovider.PermUpload},
- },
- },
- Filters: dataprovider.UserFilters{
- BaseUserFilters: sdk.BaseUserFilters{
- FilePatterns: []sdk.PatternsFilter{
- {
- Path: "/denied",
- DeniedPatterns: []string{"*.txt"},
- DenyPolicy: sdk.DenyPolicyHide,
- },
- },
- },
- },
- }),
- }
- err := connection.canReadLink("/sub/link")
- assert.ErrorIs(t, err, sftp.ErrSSHFxPermissionDenied)
- err = connection.canReadLink("/denied/file.txt")
- assert.ErrorIs(t, err, sftp.ErrSSHFxNoSuchFile)
- }
|