aoout / obsidian-epub-importer

Import EPUB files as Markdown.
MIT License
110 stars 2 forks source link

无法解析特定的epub文件 #47

Closed chengsuen closed 5 months ago

chengsuen commented 5 months ago

%%please test the bug in a black vault first. %%if there is something on console, please upload the screenshot. 屏幕截图 2024-04-01 201732

(其实是我这边手头所有的epub都没法解析

net-frog commented 5 months ago

I am having the same issue. A test epub is attached. test.zip

aoout commented 5 months ago

@net-frog What version of the plugin are you using? I didn't find this problem on 0.3.1-beta.1

aoout commented 5 months ago

Please upload .epub file.

net-frog commented 5 months ago

I have uploaded it in zip, though I am not using beta. How to update it to beta? Obsidian only have 3.0

aoout commented 5 months ago

@net-frog You can get beta through brat.

I just tested on 0.3.0 and everything works equally well.

net-frog commented 5 months ago

I have tested with the 0.3.1beta, the epub is imported successfully but all the notes are empty although each note is named as the chapter name in the epub. Also, after deleting the book folder I cannot import the same epub again, it says duplicate folder exists.

aoout commented 5 months ago

@net-frog Sorry, but I can't reproduce your situation. I retried the import again and everything worked fine and the content displayed normally.

net-frog commented 5 months ago

I have tested on 3 different systems it seems there is some underlining bug. If I have base obsidian installer 1.4.13 but have catalyst update to 1.5.12, on one system it works without issue. However, if the base installer is updated to latest 1.5.12 the following error is thrown. I have also checked with 0.3.1-beta.1 but same result.I suspect that there might be issue with tmp folder Please check with updated installer 1.5.12 but honest with every install I am getting different results.

image

aoout commented 5 months ago

@net-frog I understand, the newer installer brought newer nodejs, and one of the modules I was using had a bug on the newer nodejs, so this all happened. Reference: https://github.com/ZJONSSON/node-unzipper/issues/286

aoout commented 5 months ago

I have replaced it with another module and released version 0.3.1.

aoout commented 5 months ago

@chengsuen 请下载0.3.1版本进行尝试

aoout commented 5 months ago

I'll close the issue for now. If the bug still exists, please reopen this issue.