Merge pull request #490 from glebteterin/bug-489

Fix missing io.EOF when concurency is disabled #489
pkg · Jan 19, 2022 · 7d25d53 · 7d25d53
2 parents 31aac3e + dad2375
commit 7d25d53
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 4 deletions.
diff --git a/client.go b/client.go
@@ -999,9 +999,6 @@ func (f *File) readAtSequential(b []byte, off int64) (read int, err error) {
 			read += n
 		}
 		if err != nil {
-			if errors.Is(err, io.EOF) {
-				return read, nil // return nil explicitly.
-			}
 			return read, err
 		}
 	}

diff --git a/client_integration_test.go b/client_integration_test.go
@@ -1203,7 +1203,7 @@ func TestClientReadSequential(t *testing.T) {
 
 		stuff := make([]byte, 32)
 		n, err := sftpFile.Read(stuff)
-		require.NoError(t, err)
+		require.ErrorIs(t, err, io.EOF)
 		require.Equal(t, len(content), n)
 		require.Equal(t, content, stuff[0:len(content)])