Skip to content

Instantly share code, notes, and snippets.

@yakuter
Created April 25, 2021 19:25
Show Gist options
  • Star 4 You must be signed in to star a gist
  • Fork 1 You must be signed in to fork a gist
  • Save yakuter/6bf1e565311d11251febda4a04a6bc64 to your computer and use it in GitHub Desktop.
Save yakuter/6bf1e565311d11251febda4a04a6bc64 to your computer and use it in GitHub Desktop.
IO Copy cancellation
// Source: https://ixday.github.io/post/golang-cancel-copy/
import (
"io"
"context"
)
// here is some syntaxic sugar inspired by the Tomas Senart's video,
// it allows me to inline the Reader interface
type readerFunc func(p []byte) (n int, err error)
func (rf readerFunc) Read(p []byte) (n int, err error) { return rf(p) }
// slightly modified function signature:
// - context has been added in order to propagate cancelation
// - I do not return the number of bytes written, has it is not useful in my use case
func Copy(ctx context.Context, dst io.Writer, src io.Reader) error {
// Copy will call the Reader and Writer interface multiple time, in order
// to copy by chunk (avoiding loading the whole file in memory).
// I insert the ability to cancel before read time as it is the earliest
// possible in the call process.
_, err := io.Copy(out, readerFunc(func(p []byte) (int, error) {
// golang non-blocking channel: https://gobyexample.com/non-blocking-channel-operations
select {
// if context has been canceled
case <-ctx.Done():
// stop process and propagate "context canceled" error
return 0, ctx.Err()
default:
// otherwise just run default io.Reader implementation
return in.Read(p)
}
}))
return err
}
@maksymlumen
Copy link

out ?

@thediveo
Copy link

Doesn't work: this will still block in in.Read(p) until something becomes available to read and only afterwards checks the context.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment