diff options
| author | Jeff Halter <868228+jhalter@users.noreply.github.com> | 2022-05-31 10:43:48 -0700 |
|---|---|---|
| committer | Jeff Halter <868228+jhalter@users.noreply.github.com> | 2022-05-31 10:43:48 -0700 |
| commit | 165b7ac9486067e349fc8cc71f1d10851c8db40e (patch) | |
| tree | 901f6bf7553f6a4b4fae9cefa21185fca595e106 /hotline | |
| parent | 7778a321b14898bc8dabe4bb19c8ce5cb0100654 (diff) | |
Buffer file writes during upload
Diffstat (limited to 'hotline')
| -rw-r--r-- | hotline/transfer.go | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/hotline/transfer.go b/hotline/transfer.go index 703dae0..706dea9 100644 --- a/hotline/transfer.go +++ b/hotline/transfer.go @@ -1,6 +1,7 @@ package hotline import ( + "bufio" "bytes" "encoding/binary" "errors" @@ -28,6 +29,7 @@ func (tf *transfer) Write(b []byte) (int, error) { return len(b), nil } +const fileCopyBufSize = 524288 // 512k func receiveFile(conn io.Reader, targetFile io.Writer, resForkFile io.Writer) error { ffhBuf := make([]byte, 24) if _, err := conn.Read(ffhBuf); err != nil { @@ -75,11 +77,14 @@ func receiveFile(conn io.Reader, targetFile io.Writer, resForkFile io.Writer) er // this will be zero if the file only has a resource fork fileSize := int(binary.BigEndian.Uint32(ffdfh.DataSize[:])) - // Copy N bytes from conn to upload file - _, err = io.CopyN(targetFile, conn, int64(fileSize)) + bw := bufio.NewWriterSize(targetFile, fileCopyBufSize) + _, err = io.CopyN(bw, conn, int64(fileSize)) if err != nil { return err } + if err := bw.Flush(); err != nil { + return err + } if ffh.ForkCount == [2]byte{0, 3} { var resForkHeader FlatFileDataForkHeader @@ -95,10 +100,14 @@ func receiveFile(conn io.Reader, targetFile io.Writer, resForkFile io.Writer) er fileSize = int(binary.BigEndian.Uint32(resForkHeader.DataSize[:])) + bw = bufio.NewWriterSize(resForkFile, fileCopyBufSize) _, err = io.CopyN(resForkFile, conn, int64(fileSize)) if err != nil { return err } + if err := bw.Flush(); err != nil { + return err + } } return nil } |