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.

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

Re-enable Windows_ZipWithInvalidFileNames tests #55434

Merged
merged 1 commit into from
Jul 10, 2021

Conversation

eerhardt
Copy link
Member

@eerhardt eerhardt commented Jul 9, 2021

Since we have shipped for 3 years with the current behavior, re-enabling the test so it ensures the current behavior going forward. Extracting .zip files with invalid file names throw an IOException with a message "The filename, directory name, or volume label syntax is incorrect.".

Contributes to #25099

Since we have shipped for 3 years with the current behavior, re-enabling the test so it ensures the current behavior going forward. Extracting .zip files with invalid file names throw an IOException with a message "The filename, directory name, or volume label syntax is incorrect.".

Contributes to dotnet#25099
@ghost
Copy link

ghost commented Jul 9, 2021

Tagging subscribers to this area: @dotnet/area-system-io-compression
See info in area-owners.md if you want to be subscribed.

Issue Details

Since we have shipped for 3 years with the current behavior, re-enabling the test so it ensures the current behavior going forward. Extracting .zip files with invalid file names throw an IOException with a message "The filename, directory name, or volume label syntax is incorrect.".

Contributes to #25099

Author: eerhardt
Assignees: -
Labels:

area-System.IO.Compression

Milestone: -

@eerhardt
Copy link
Member Author

The test failures are http functional tests, unrelated to this change.

@eerhardt eerhardt merged commit ecea9a3 into dotnet:main Jul 10, 2021
@eerhardt eerhardt deleted the FixCompressionTest branch July 10, 2021 02:03
@ghost ghost locked as resolved and limited conversation to collaborators Aug 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants