freedomofpress / dangerzone

Take potentially dangerous PDFs, office documents, or images and convert them to safe PDFs
https://dangerzone.rocks/
GNU Affero General Public License v3.0
3.59k stars 170 forks source link

Temporarily pin PyMuPDF==1.23.8 in container #703

Closed deeplow closed 7 months ago

deeplow commented 7 months ago

PyMuPDF 1.23.9 made the swapped the new fitz implementation (fitz_new) with the fitz module. In the new module there are prints in the code that interfere with our stderror for sending JSON from the container. Pinning the version seems to have no adverse consequences, since fitz_old hasn't had significant changes and it gives breething room for the print-related issue to be tackled in this PR.

Fixes temporarily #700

apyrgio commented 7 months ago

Some fixes in the commit message:

made the swapped the

swapped the

with our stderror

with our stdout

breething room

breathing room

deeplow commented 7 months ago

Merging this. The failing should be unrelated to this PR (they are also happening on main)