Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[release/7.0] TarReader should dispose underlying stream if leaveOpen is false #80598

Merged
merged 3 commits into from
Jan 13, 2023
Merged

[release/7.0] TarReader should dispose underlying stream if leaveOpen is false #80598

merged 3 commits into from
Jan 13, 2023

Commits on Jan 12, 2023

  1. TarReader should dispose underlying stream if leaveOpen is false (dot… 

    …net#79899)
    @jozkee
    jozkee committed Jan 12, 2023
    Configuration menu
    Copy the full SHA
    08e8e19 View commit details
    Browse the repository at this point in the history

  2. Dispose underlying stream in TarReader.DisposeAsync() as well (dotnet… 

    …#79920)

* Dispose underlying stream in TarReader.DisposeAsync() as well

Same as dotnet#79899

* Consolidate duplicated WrappedStream test helpers to Common sources

* Dispose stream passed to WrappedStream
    @akoeplinger @jozkee
    akoeplinger authored and jozkee committed Jan 12, 2023
    Configuration menu
    Copy the full SHA
    1fbde51 View commit details
    Browse the repository at this point in the history

Commits on Jan 13, 2023

  1. Dispose archive stream after the list of DataStreams (dotnet#80572) 

    * Dispose archive stream after the list of DataStreams

* Add tests for TarReader.DisposeAsync properly disposing underlying stream
    @jozkee
    jozkee committed Jan 13, 2023
    Configuration menu
    Copy the full SHA
    b1f9f65 View commit details
    Browse the repository at this point in the history