adm-zip icon indicating copy to clipboard operation
adm-zip copied to clipboard

doesn't support write zip64 file

Open hkbarton opened this issue 4 years ago • 6 comments

seems like this library is still leaking of zip64 support. I saw there is a PR from @pierre-lehnen-rc https://github.com/cthackers/adm-zip/pull/301 to address zip64 support but seems like it only supports to read zip64 file. When saving a zip file with more than 65535 entries, the code here https://github.com/cthackers/adm-zip/blob/master/headers/mainHeader.js#L67 is still not working, it's assuming writing normal zip file and will throw buffer error here.

hkbarton avatar Jul 15 '20 08:07 hkbarton

Still no solution -_-

BadMachine avatar May 21 '21 15:05 BadMachine

@5saviahv @cthackers Are you planning to support this feature soon? If not I am considering to do some investigation and bring a PR to add this feature

timqian avatar May 28 '21 06:05 timqian

I had a plan to add this feature, I have even some pre-alpha stage code for it.

5saviahv avatar May 29 '21 15:05 5saviahv

@5saviahv Sounds great, thanks~

timqian avatar May 31 '21 01:05 timqian

@5saviahv Looking for get this fixed , please update the issue and let me know when it is released. thx.

ole3021 avatar Jun 17 '21 07:06 ole3021

@5saviahv any update on this issue?

ole3021 avatar Mar 29 '22 14:03 ole3021