Skip to content
This repository has been archived by the owner on Jul 18, 2024. It is now read-only.

All path separators needs to be normalized to forward slash #48

Closed
SteveL-MSFT opened this issue Nov 15, 2017 · 1 comment
Closed

All path separators needs to be normalized to forward slash #48

SteveL-MSFT opened this issue Nov 15, 2017 · 1 comment
Labels
Issue-bug

Comments

@SteveL-MSFT
Copy link
Member

Even on Windows, the archive contents should only use forward slash as directory separator otherwise it causes problems on non-Windows systems where backslash is a valid character in a filename.

According to the Zip file spec section 4.4.17.1, the path MUST only use forward slashes
@SteveL-MSFT SteveL-MSFT mentioned this issue Nov 16, 2017
Closed
@mryanmurphy mryanmurphy mentioned this issue Feb 8, 2019
Merged
@anmenaga
Copy link
Collaborator

Fixed in module version 1.2.3.0
Install-Module Microsoft.PowerShell.Archive -MinimumVersion 1.2.3.0 -Repository PSGallery -Force

@Algorithman Algorithman mentioned this issue May 9, 2021
Closed
3b added a commit to 3b/zip that referenced this issue Dec 13, 2021
@3b
convert (invalid) #\\ to #\/ in UNZIP
e026b96 
some .NET and powershell versions create invalid files using \ as path separator, so try to handdle them as expected.

PowerShell/Microsoft.PowerShell.Archive#48
https://docs.microsoft.com/en-us/dotnet/framework/migration-guide/mitigation-ziparchiveentry-fullname-path-separator
@bsweeney bsweeney mentioned this issue Jan 10, 2022
Closed
@dignissimus dignissimus mentioned this issue May 16, 2022
Open
@ahal ahal mentioned this issue Jul 29, 2022
Closed
@Hiroshiba Hiroshiba mentioned this issue Jan 9, 2023
Closed
@sarsharma sarsharma mentioned this issue Jun 20, 2023
Closed
@HebaruSan HebaruSan mentioned this issue Sep 21, 2023
Merged
@Hr0bar Hr0bar mentioned this issue Sep 22, 2023
Closed
@HebaruSan HebaruSan mentioned this issue Oct 23, 2023
Merged
@fliiiix fliiiix mentioned this issue Mar 22, 2024
Closed
@ziafazal ziafazal mentioned this issue Jun 28, 2024
Merged
@mo271 mo271 mentioned this issue Sep 16, 2024
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Issue-bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@SteveL-MSFT @anmenaga