Merge pull request #59 from Mirio/develop

Adding urlparser for sanification the input

README.md

@@ -20,7 +20,7 @@ With VerbaCap is a **Podcast Manager** you will be able to download and listen t
 It uses [_Django_](https://www.djangoproject.com/) in order to create a new integration for the platform as easy as possible. Below a quick platform integrated and the current status.
 
 :heavy_check_mark: Youtube Channel :heavy_check_mark: Youtube Playlist
-:construction: Spreaker.com
+:heavy_check_mark: Spreaker.com
 :construction: Apple Podcast :construction: Amazon Music
 
 [Installation](docs/install.md) • [Configuration](docs/config.md)

spreaker/services.py

@@ -1,4 +1,5 @@
 from os.path import exists
+from urllib.parse import urlparse
 
 import feedparser
 import requests
@@ -10,7 +11,8 @@
 
 def get_rssurl(input_url: str) -> CommonResponse:
     out = CommonResponse()
-    if not input_url.startswith("https://www.spreaker.com/show/"):
+    inputurl_parsed = urlparse(input_url)
+    if not inputurl_parsed.path.startswith("/show") or inputurl_parsed.hostname != "www.spreaker.com":
         out.status = "error"
         out.message = "Not a spreaker url"
     else:
@@ -29,7 +31,8 @@ def get_rssurl(input_url: str) -> CommonResponse:
 def get_rss_data(input_url: str, limit: int = 10) -> CommonResponse:
     out = CommonResponse()
     counter = 0
-    if not input_url.startswith("https://www.spreaker.com/show/"):
+    inputurl_parsed = urlparse(input_url)
+    if not inputurl_parsed.path.startswith("/show") or inputurl_parsed.hostname != "www.spreaker.com":
         out.status = "error"
         out.message = "Not a spreaker url"
     else:
@@ -45,7 +48,8 @@ def get_rss_data(input_url: str, limit: int = 10) -> CommonResponse:
 
 def get_audio(input_url: str, fname: str) -> CommonResponse:
     out = CommonResponse()
-    if not input_url.startswith("https://dts.podtrac.com") and not input_url.startswith("https://api.spreaker.com"):
+    inputurl_parsed = urlparse(input_url)
+    if inputurl_parsed.hostname != "api.spreaker.com" and inputurl_parsed.hostname != "dts.podtrac.com":
         out.status = "error"
         out.message = "Not a spreaker url"
     else:

youtube/services.py

@@ -1,5 +1,5 @@
 from os.path import exists
-from urllib.parse import parse_qs
+from urllib.parse import parse_qs, urlparse
 
 import feedparser
 import requests
@@ -13,7 +13,8 @@
 
 def get_channel_rssurl(input_url: str) -> CommonResponse:
     out = CommonResponse()
-    if not input_url.startswith("https://www.youtube.com/"):
+    inputurl_parsed = urlparse(input_url)
+    if inputurl_parsed.hostname != "www.youtube.com":
         out.status = "error"
         out.message = "Not a youtube url"
     else:
@@ -32,7 +33,8 @@ def get_channel_rssurl(input_url: str) -> CommonResponse:
 
 def get_playlist_rssurl(input_url: str) -> CommonResponse:
     out = CommonResponse()
-    if not input_url.startswith("https://www.youtube.com/"):
+    inputurl_parsed = urlparse(input_url)
+    if inputurl_parsed.hostname != "www.youtube.com":
         out.status = "error"
         out.message = "Not a youtube url"
     elif "&list=" not in input_url:
@@ -47,9 +49,10 @@ def get_playlist_rssurl(input_url: str) -> CommonResponse:
 
 def get_rss_data(input_url: str, limit: int = 10) -> CommonResponse:
     out = CommonResponse()
+    inputurl_parsed = urlparse(input_url)
     counter = 0
     if input_url:
-        if not input_url.startswith("https://www.youtube.com/"):
+        if inputurl_parsed.hostname != "www.youtube.com":
             out.status = "error"
             out.message = "Not a youtube url"
         else:
@@ -68,7 +71,8 @@ def get_rss_data(input_url: str, limit: int = 10) -> CommonResponse:
 
 def get_audio(input_url: str, fname: str) -> CommonResponse:
     out = CommonResponse()
-    if not input_url.startswith("https://www.youtube.com/"):
+    inputurl_parsed = urlparse(input_url)
+    if inputurl_parsed.hostname != "www.youtube.com":
         out.status = "error"
         out.message = "Not a youtube url"
     else: