Covers of songs have `EOI missing` (according to ffprobe)
javalsai opened this issue · comments
Describe the bug
Most of the covers of the songs have EOI
missing (according to ffprobe), which means that the image is corrupt or didn't finish downloading when being added to the song. This leads to the image not displaying correctly on file managers, music players...
To Reproduce
Steps to reproduce the behavior:
- Download any playlist with the spotify-dl cli and most images should be like this.
Expected behavior
Images fine added to the song.
Desktop (please complete the following information):
- OS: Arch Linux (5.18.14-arch1-1)
- Browser Firefox (Mozilla Firefox 103.0)
- spotify-dl version: 1.0.8
Not sure if it is because of this, but assuming that the cover is a jpg
isn't probably a good idea.
Lines 8 to 31 in 09b803f
feel free to give png
a try, from memory it had to be a jpg
but i could be wrong
@phyzical I see that for audio download you use ffmpeg to make sure of the format, as far as I know ffmpeg also works with images, so I'm giving it a try
Seems to be working, I'll download a full playlist and if it does work I'll open a PR