-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Doesn't work with Zip64 files #834
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
Comments
First, this repo is for feedback on the documentation, so I think this issue doesn't belong here, unless you think this is something that should be pointed out in the documentation. Second, as far as I can tell from https://github.com/dotnet/corefx/issues/9925, .Net Core does support Zip64 files. .Net Core improvements are sometimes ported to .Net Framework, though I'm not sure how likely is that in this case. Though it might still make sense to ask. If you still want to provide feedback on .Net Framework, the right place is Developer Community. For .Net Core APIs, the right place is the corefx repo (but I don't think this is a .Net Core issue). |
@ianhays can you help with this question? Is there something we should add to the docs? Thanks! |
Yes it should state in the documentation that it doesn't support Zip64, Nowadays it is common to find such large files, would be helpful also if it stays what future frameworks (if any) would support Zip64 .. It is not simple to switch from regular .Net to .Net core in exiting products |
@svick is correct. There is also a distinction between zip64 and deflate64 and neither of them are exclusive with size limitations:
Since we do support entries larger than 4gb, we shouldn't make any doc changes. We should probably backport Deflate64 support as a functional improvement, but that's a separate issue that should be filed elsewhere as @svick mentioned. |
I think this was answered a long time ago, so we should close it. |
The ZipArchive can't read any entry that is larger than 4,294,967,295 bytes .. which is the old limit for zip files although the OS can read this.. would appreciate if such capabilities is enabled instead of using other packages
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.
The text was updated successfully, but these errors were encountered: