alexvanin/restic
1
0
Fork 0
forked from TrueCloudLab/restic
Code Pull requests Activity

Merge pull request #1398 from armhold/sftp-readdir

Browse source 
sftp ReadDir: add path to return error messages (gh-1323)
This commit is contained in:
Alexander Neumann 2017-10-29 09:26:42 +01:00
commit ca3cadef5e
1 changed files with 9 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
internal/backend/sftp/sftp.go
View file

@ -146,7 +146,7 @@ func (r *SFTP) checkDataSubdirs() error {
datadir := r.Dirname(restic.Handle{Type: restic.DataFile})
// check if all paths for data/ exist
entries, err := r.c.ReadDir(datadir)
entries, err := r.ReadDir(datadir)
if r.IsNotExist(err) {
return nil
}
@ -193,7 +193,12 @@ func (r *SFTP) Join(p ...string) string {
// ReadDir returns the entries for a directory.
func (r *SFTP) ReadDir(dir string) ([]os.FileInfo, error) {
return r.c.ReadDir(dir)
fi, err := r.c.ReadDir(dir)
// sftp client does not specify dir name on error, so add it here
err = errors.Wrapf(err, "(%v)", dir)
return fi, err
}
// IsNotExist returns true if the error is caused by a not existing file.
@ -476,7 +481,7 @@ var closeTimeout = 2 * time.Second
// Close closes the sftp connection and terminates the underlying command.
func (r *SFTP) Close() error {
debug.Log("")
debug.Log("Close")
if r == nil {
return nil
}
@ -501,7 +506,7 @@ func (r *SFTP) Close() error {
}
func (r *SFTP) deleteRecursive(name string) error {
entries, err := r.c.ReadDir(name)
entries, err := r.ReadDir(name)
if err != nil {
return errors.Wrap(err, "ReadDir")
}