-
Notifications
You must be signed in to change notification settings - Fork 114
Unzip issues on i2pseeds.su3 file #795
Comments
And refactor/redesign/rewrite the unzip part may help #785 |
I cannot reproduce either issue using the same su3 file. Please attach a log with |
@wqking can you please upload the following:
Thank you. |
I uploaded the log files. kovri-build.txt is the build information, kovri-log.txt is the log. |
Please build against current master or d6a4851, whichever is newer by the time you read this. Also, the file doesn't show complete build information. Please paste complete build information as well as
This doesn't show complete log information. Please paste complete log information from router start to end. The su3 file you're using is now stale. If you can, please grab a fresh one from one of the hardcoded reseed servers that is not https://i2p.novg.net/, using the following command or user-agent: |
I built Kovri on Ubuntu 16.0.4 (VirtualBox on my Win10) and it doesn't have any error with the su3 file. |
Since I can't reproduce on win10 msys2/mingw (English locale), this is may be the case. If you provided logs, I can help debug. |
NOTICE: THIS ISSUE HAS BEEN MOVED TO GitLab. Please continue the discussion there. See #1013 for details. |
OS: Win10. Kovri was built using MSys2, MinGW64.
I download the seeds file here,
https://i2p.novg.net/i2pseeds.su3
I also attached the file.
i2pseeds.su3.zip
Issue 1:
Command
kovri.exe --reseed-from i2pseeds.su3
The memory usage jump between 1G and 4G very quickly, and after some long time, kovri gives some zip related error.
Issue 2
Command
kovri.exe --enable-su3-verification=0 --reseed-from i2pseeds.su3
kovri gives zip related error right away.
[error] ZIP: CRC-32 Failed
I also use WinRar to unzip the .su3, then compress it using standard zip method, both issues 1 & 2 exist.
My thoughts: Kovri uses its own version unzip code, seems it doesn't support the zip standard well.
If the issue is confirmed, I'm happy to work on it. We may replace the zip module with existing mature zip library, etc.
By submitting this issue, I confirm the following:
The text was updated successfully, but these errors were encountered: