Flesh out `ByteRangeStream` methods to make it always limit reads

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

Closed dougbu closed 5 years ago

dougbu commented 6 years ago

206
Seek(...) was a no-op, Position was incorrect if _lowerbounds != 0, ReadAsync(...) read entire inner Stream
- rewrite ByteRangeStreamTest to cover new ByteRangeStream members and hold fewer resources
remove DelegatingStream.CopyToAsync(...) override because it copied entire inner Stream and was not needed
- base implementation invokes ReadAsync(...)

dougbu commented 6 years ago

FYI @stephentoub

dougbu commented 5 years ago

:up::date: to rebase and address @mkArtakMSFT's comment. Will merge once the CI check passes.