jeans/core
jeans/core
1
0
Fork 0
mirror of https://github.com/home-assistant/core.git synced 2025-07-27 07:07:28 +00:00
Code Issues Packages Projects Releases Wiki Activity

Handle query and anchors in Spotify URI's (#33084)

Browse Source 
* Handle query and anchors in Spotify URI's

* Use yarl for cleaning up the URL
This commit is contained in:
Franck Nijhof 2020-03-21 20:24:23 +01:00 committed by GitHub
parent 699ca44260
commit 8272c71811
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
homeassistant/components/spotify/media_player.py
View File

@ -7,6 +7,7 @@ from typing import Any, Callable, Dict, List, Optional
from aiohttp import ClientError from aiohttp import ClientError
from spotipy import Spotify, SpotifyException from spotipy import Spotify, SpotifyException
from yarl import URL
from homeassistant.components.media_player import MediaPlayerDevice from homeassistant.components.media_player import MediaPlayerDevice
from homeassistant.components.media_player.const import ( from homeassistant.components.media_player.const import (
@ -295,6 +296,10 @@ class SpotifyMediaPlayer(MediaPlayerDevice):
"""Play media.""" """Play media."""
kwargs = {} kwargs = {}
# Spotify can't handle URI's with query strings or anchors
# Yet, they do generate those types of URI in their official clients.
media_id = str(URL(media_id).with_query(None).with_fragment(None))
if media_type == MEDIA_TYPE_MUSIC: if media_type == MEDIA_TYPE_MUSIC:
kwargs["uris"] = [media_id] kwargs["uris"] = [media_id]
elif media_type == MEDIA_TYPE_PLAYLIST: elif media_type == MEDIA_TYPE_PLAYLIST: