search

aspnet / AspNetWebStack

ASP.NET MVC 5.x, Web API 2.x, and Web Pages 3.x (not ASP.NET Core)
Other
858 stars 354 forks source link

`ByteRangeStream` does not limit `CopyToAsync(...)` range as it should #206

Closed dougbu closed 6 years ago

dougbu commented 6 years ago

Fix is to remove the CopyToAsync(...) override in DelegatingStream. The underlying Stream's implementation of this method will do the Right Thing:tm: and use methods that are correctly overridden in DelegatingStream and its subclasses, including ByteRangeStream.

Issue originally tracked in https://devdiv.visualstudio.com/DevDiv/_workitems/edit/690342

dougbu commented 6 years ago

39d3064baf