Illustration does not respect the openZIM specification

openzim / youtube

Create a ZIM file from a Youtube channel/username/playlist

GNU General Public License v3.0

39 stars 26 forks source link

Illustration does not respect the openZIM specification #199

Closed kelson42 closed 1 month ago

kelson42 commented 3 months ago

See for example: https://farm.openzim.org/pipeline/a021a6b8-019c-498a-b435-f08a3dedac19

Zimcheck report: "file_name": "/data/nota-bene_fr_all_2024-03.zim", "file_uuid": "1efae61e-9ee7-271d-9120-4bfa0fd892de", "logs": [ { "check": "metadata", "error": "Illustration_48x48@1 doesn't match regex: ^\x89PNG\x0d\x0a\x1a\x0a", "level": "ERROR", "message": "Illustration_48x48@1 doesn't match regex: ^\x89PNG\x0d\x0a\x1a\x0a" },

rgaudin commented 3 months ago

It's a JPEG image…

kelson42 commented 1 month ago

If I got is right, using latest scraperlib would help to get that fixed straight, see #175

rgaudin commented 1 month ago

Yes, the run would have failed ; and there are conversion functions to use

benoit74 commented 1 month ago

Fixed by https://github.com/openzim/youtube/pull/202