Eagerly cancel timeout context when response has no body

Azure / azure-storage-blob-go

Microsoft Azure Blob Storage Library for Go

MIT License

157 stars 102 forks source link

Eagerly cancel timeout context when response has no body #309

Closed serprex closed 3 years ago

serprex commented 3 years ago

Fixes #253

@ncw or @patyupin are you able to test this?

I'm not sure how to build/test, go build outputs no Go files in /home/.../azure-storage-blob-go

mohsha-msft commented 3 years ago

Running CI.

ncw commented 3 years ago

I have tested this pull request in rclone by uploading 200,000 files. I can report that there was no memory leak and context.WithDeadline did not appear on the memory trace :-)

So, yes, this appears to fix the problem - thank you :-)

patyupin commented 2 years ago

what version of the rclone has this fix implemented?