Skip to content

Commit

Permalink
Merge pull request #2 from iRet/master
Browse files Browse the repository at this point in the history
Fix embedded cover art
  • Loading branch information
scaronni authored Mar 16, 2021
2 parents ad496e5 + b636c16 commit 121c970
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 10 deletions.
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ def _read(fn):

setup(
name='spotify-ripper',
version='2.15',
version='2.17',
packages=find_packages(exclude=["tests"]),
#scripts=['spotify_ripper/main.py'],
include_package_data=True,
Expand Down
22 changes: 13 additions & 9 deletions spotify_ripper/tags.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
import os
import sys
import base64
import urllib.request


def set_metadata_tags(args, audio_file, idx, track, ripper):
Expand Down Expand Up @@ -65,16 +66,19 @@ def set_metadata_tags(args, audio_file, idx, track, ripper):
genres_ascii = [to_ascii(genre) for genre in genres]

# cover art image
image = None
if args.large_cover_art:
image = ripper.web.get_large_coverart(track.link.uri)
def get_cover_image(image_link):
image_link = 'https://i.scdn.co%s' % (
image_link[len('spotify'):].replace(':', '/'))
cover_file = urllib.request.urlretrieve(image_link)[0]

with open(cover_file, "rb") as f:
if f.mode == "rb":
return f.read()
else:
return None

# if we fail, use regular cover size
if image is None:
image = track.album.cover()
if image is not None:
image.load(args.timeout)
image = image.data
image_link = str(track.album.cover(2).link)
image = get_cover_image(image_link)

def tag_to_ascii(_str, _str_ascii):
return _str if args.ascii_path_only else _str_ascii
Expand Down

0 comments on commit 121c970

Please sign in to comment.