Skip to content
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

Error parsing Shareaza Library1/2.dat files #1544

Closed
wladimirleite opened this issue Feb 25, 2023 · 2 comments · Fixed by #1545
Closed

Error parsing Shareaza Library1/2.dat files #1544

wladimirleite opened this issue Feb 25, 2023 · 2 comments · Fixed by #1545
Assignees
Labels

Comments

@wladimirleite
Copy link
Member

@felipecampanini is working on a case in which there are Shareaza Library1.dat and Library2.dat files.
Processing with IPED 4.1.0, 4.0.x and 3.18.x these files weren't parsed.

Investigating the issue, I found out that using the Python parser written by @fmpfeifer, both files were parsed successfully.
Comparing IPED Java parser code and the Python one, I found out that last fixes applied in the original project weren't reflected in IPED:
fmpfeifer/ShareazaParser@f220278

As the changes were very simple, I applied them to our project and confirmed the issue was fixed.
Other sample Shareaza files that I have were also processed successfully.

@fmpfeifer
Copy link
Member

Thanks @tc-wleite, it was in my backlog of things to do in iped. Unfortunately I'm busy in other projects and could not spare some time to work on this. I still have a few other items in my iped to do list that I hope I can put forward soon.
Sorry for the the long delay and thanks again

@lfcnassif
Copy link
Member

Thank you @tc-wleite for investigating!

@fmpfeifer feel free to use this issue tracker to document your TODO list. If they are bug fixes, it's important to document here. Please remember to open them here before start working to avoid duplicate efforts.

If both of you need time to work on something important, let me know and I can try to negotiate with my chief and yours to borrow some time from you, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants