Hang on viewing epub about epy

[wustho]

Wow, thanks for detailed traceback and solution suggestion, mate... Will look into that.

I just realized it should've been explicit zlib.error exception to catch...

[meganleewebb]

Unzip -t :

testing: mimetype                 OK
    testing: jacket.xhtml             OK
    testing: META-INF/container.xml   OK
    testing: OEBPS/9780062430052_toc.ncx   OK
    testing: OEBPS/9780062430052_content.opf   OK
    testing: OEBPS/images/cover.jpg   OK
    testing: OEBPS/images/auth.jpg    OK
    testing: OEBPS/images/copy.jpg    OK
    testing: OEBPS/images/copy1.jpg   OK
    testing: OEBPS/images/title.jpg   OK
    testing: OEBPS/images/heard.jpg   OK
    testing: OEBPS/imgbackad/imgbackad-1.jpg   OK
    testing: OEBPS/imgbackad/imgbackad-2.jpg   OK
    testing: OEBPS/styles/9780062430052_template.css   OK
    testing: OEBPS/text/titlepage.xhtml   OK
    testing: OEBPS/text/nav.xhtml     OK
    testing: OEBPS/text/About_the_Author.xhtml   OK
    testing: OEBPS/text/About_the_Publisher.xhtml   OK
    testing: OEBPS/text/9780062430052_Also_by.xhtml   OK
    testing: OEBPS/text/9780062430052_Credits.xhtml   OK
...

jacket.xhtml does exist in the epub.
First file accessed was titlepage.xhtml. jacket.xhtml was 2nd.

I didn't look further if the file path "OEBPS/../jacket.xhtml" should have been resolved prior to this function.

Thanks for great app. 👍

[wustho]

Hey, there just fixed this with: 50dd4fa

Can you try upgrading epy (pip install --upgrade epy-reader) and let me know if the issue still persists. Thanks.

[meganleewebb]

That works.

I think your leaving a bug by not putting an upper limit on max_tries to exit the while True loop.

max_tries: Optional[int] = None  # 1 if DEBUG else None

        # use try-except block to catch
        # zlib.error: Error -3 while decompressing data: invalid distance too far back
        # seems like caused by multiprocessing
        tries = 0
        while True:
            try:
                content = self.file.open(content_path).read()
                break
            except zlib.error as e:
                tries += 1
                if max_tries is not None and tries >= max_tries:
                    raise e

That will loop indefinitely on any zip error.

The comment:

# use try-except block to catch
# zlib.error: Error -3 while decompressing data: invalid distance too far back
# seems like caused by multiprocessing

Suggests the loop is there as more than one attempt is needed for that error. But some upper limit would seem wise.

I've not seen that error, but I have seen zip files that have one member corrupted and that member is unreadable, while the rest are.

[wustho]

Oh, that's indeed wise idea! Putting it in TODO, thanks!