-
Notifications
You must be signed in to change notification settings - Fork 6
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
Can't Use Embedded ZIP File #4
Comments
@eXpl0it3r I'm quite busy these days. Could you try your solution and propose a pull request ? I will be happy to merge it, if you succeed. Don't forget a minimal unit test. |
@eXpl0it3r I've just published a new nuget version (1.0.7) with your PR. I let you close this issue |
Oh awesome, that was quick, thanks a lot! 🙂 I'll write a test and check that the implementation fully works, afterwards close the issue.
|
thanks for you PR #6 , I've just merge it. |
Thanks again for your quick response! 🙂 As mentioned in #7, you may want to publish a new NuGet package, since I broke the standard NetZipArchiveFileSystem, as it will throw when trying to open an entry as ReadWrite, but I'll close this issue. |
Switching from SharpZipLib to ZipArchive made it a requirement for NetZipArchiveFileSystem that the Stream is not only readable (and seekable), but also writeable, which doesn't work when you take the stream of a read-only embedded resource file.
Would there be a possibility to add support to read-only ZIP files as well?
I'd guess that having an additional function
OpenReadOnly(Stream s)
which then opens the file withZipArchiveMode.Read
should be enough. Maybe add an additional check to the write methods to throw an exception in ReadOnly mode.The text was updated successfully, but these errors were encountered: