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

Unable to parse DZI file with a Byte Order Mark #45

Closed
macmacleannz opened this issue May 19, 2020 · 11 comments
Closed

Unable to parse DZI file with a Byte Order Mark #45

macmacleannz opened this issue May 19, 2020 · 11 comments
Labels
bug Something isn't working

Comments

@macmacleannz
Copy link

I am getting the following error using dezoomify-rs

ERROR Dezoomer error: Unable to create the dezoomer: Tried all of the dezoomers, none succeeded. They returned the following errors:

 - deepzoom: Unable to create the dezoomer: Unable to parse the dzi file: Syntax: 1:1 Unexpected characters outside the root element: 

Using the same URL I have no problem using Dezoomify on the web

http://www.cartomundi.fr/site/cmfiles/Cartes/SP8/d_Eg-124.jpg/dzc_output.xml

Any clues on what I am doing wrong?

Thanks
Mac

@lovasoa lovasoa added the bug Something isn't working label May 19, 2020
@lovasoa
Copy link
Owner

lovasoa commented May 19, 2020

Hello, and thank you for reporting this. After investigation, I found the origin of the bug. It is in the underlying XML parser library dezoomify-rs uses: RReverser/serde-xml-rs#67

So, this issue will be resolved when RReverser/serde-xml-rs#67 is.

@lovasoa lovasoa changed the title Unable to parse DZI file Unable to parse DZI file with a Byte Order Mark May 19, 2020
lovasoa added a commit that referenced this issue May 19, 2020
lovasoa added a commit that referenced this issue May 19, 2020
Log metadata file contents when in debug mode.
@lovasoa
Copy link
Owner

lovasoa commented May 19, 2020

It looks like the developer of the underlying library is reluctant to fix this bug: netvl/xml-rs#10

@macmacleannz
Copy link
Author

Thanks for the prompt response! Do I have any other options to undertake multiple downloads? dezoomify-rs has been great for creating batch downloads.

@lovasoa
Copy link
Owner

lovasoa commented May 20, 2020

I can add a workaround for this bug... In the meantime, there is still the node version of dezoomify

@lovasoa
Copy link
Owner

lovasoa commented May 20, 2020

OK, I implemented a workaround for the underlying issue. I'll soon release a new version of dezoomify-rs, and you'll be able to use it with your file.

@lovasoa
Copy link
Owner

lovasoa commented May 20, 2020

You can download the latest build of dezoomify-rs with the fix here: https://github.com/lovasoa/dezoomify-rs/releases/tag/untagged-7390da2b90bad1bc3e9a

@macmacleannz
Copy link
Author

Fantastic! I get a 404 when clicking on the link.

@lovasoa
Copy link
Owner

lovasoa commented May 20, 2020

Here is the right link : https://github.com/lovasoa/dezoomify-rs/releases/tag/build-25e9c8d18aa0196c59d5e2050bd6fb3443b9fcd5
I didn't realize the release wasn't public.

@macmacleannz
Copy link
Author

Sorry to be a pain - still getting a 404....

@macmacleannz
Copy link
Author

Ok got it now. Will let you know it works

@macmacleannz
Copy link
Author

Perfect - works as expected. Thanks for resolving this bug so promptly!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants