Skip to content

Conversation

@iremyux
Copy link
Contributor

@iremyux iremyux commented Sep 23, 2025

This PR improves the experience of reading compressed tar archives by detecting common compression formats before parsing. It now provides clear, localized messages that explain whether the archive is compressed with a supported format (e.g., GZIP, ZLIB) or an unsupported one (e.g., BZIP2, LZ4, XZ). For supported formats, it guides users to wrap the stream with the right .NET class, and for unsupported formats, it advises decompression first.

Fixes #89056

Copilot AI review requested due to automatic review settings September 23, 2025 14:41
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR enhances the System.Formats.Tar library by adding compression format detection at the beginning of tar archive reading. When users attempt to read a compressed tar file directly without decompression, the library now provides helpful error messages that identify the compression type and guide users on the appropriate solution.

  • Adds magic number detection for common compression formats (GZIP, ZLIB, BZIP2, LZ4, XZ, etc.)
  • Provides differentiated error messages for supported vs unsupported compression formats
  • Integrates compression checking into both synchronous and asynchronous tar reading workflows

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
TarHeader.Read.cs Adds compression detection logic and calls to check magic numbers during tar header reading
Strings.resx Adds localized error message templates for supported and unsupported compression detection

@iremyux iremyux requested a review from Copilot October 27, 2025 11:47
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tar: Detect magic numbers in archives compressed with popular algorithms

1 participant