Failsafe in case channel 10 returns bad results again

[webdl] / sbs.py

diff --git a/sbs.py b/sbs.py
index 043e63045f2f5e05dc04305011db2b71505a4905..e2326ee513ccf36a4d7055e524a51b58cfb0730d 100644 (file)
--- a/sbs.py
+++ b/sbs.py
@@ -2,9 +2,11 @@ import requests_cache
 from common import grab_html, grab_json, grab_xml, download_hls, download_mpd, Node, append_to_qs
 
 import json
+import logging
+import sys
 
 BASE = "https://www.sbs.com.au"
-FULL_VIDEO_LIST = BASE + "/api/video_search/v2/?m=1&filters={section}{Programs}"
+FULL_VIDEO_LIST = BASE + "/api/video_feed/f/Bgtm9B/sbs-section-programs/"
 VIDEO_URL = BASE + "/ondemand/video/single/%s"
 
 NS = {
@@ -22,8 +24,13 @@ class SbsVideoNode(Node):
         with requests_cache.disabled():
             doc = grab_html(VIDEO_URL % self.video_id)
         player_params = self.get_player_params(doc)
-        release_url = player_params["releaseUrls"]["html"]
 
+        error = player_params.get("error", None)
+        if error:
+            print("Cannot download:", error)
+            return False
+
+        release_url = player_params["releaseUrls"]["html"]
         filename = self.title + ".ts"
 
         hls_url = self.get_hls_url(release_url)
@@ -76,18 +83,37 @@ class SbsRootNode(SbsNavNode):
 
     def load_all_video_entries(self):
         offset = 1
-        amount = 500
+        page_size = 500
+        results = {}
+        duplicate_warning = False
+
         while True:
-            url = append_to_qs(FULL_VIDEO_LIST, {"range": "%s-%s" % (offset, offset+amount)})
-            data = grab_json(url)
-            if "entries" not in data:
-                raise Exception("Missing data in SBS response", data)
-            entries = data["entries"]
+            entries = self.fetch_entries_page(offset, page_size)
             if len(entries) == 0:
                 break
+
             for entry in entries:
-                yield entry
-            offset += amount
+                guid = entry["guid"]
+                if guid in results and not duplicate_warning:
+                    # https://bitbucket.org/delx/webdl/issues/102/recent-sbs-series-missing
+                    logging.warn("SBS returned a duplicate response, data is probably missing. Try decreasing page_size.")
+                    duplicate_warning = True
+
+                results[guid] = entry
+
+            offset += page_size
+            sys.stdout.write(".")
+            sys.stdout.flush()
+
+        print()
+        return list(results.values())
+
+    def fetch_entries_page(self, offset, page_size):
+        url = append_to_qs(FULL_VIDEO_LIST, {"range": "%s-%s" % (offset, offset+page_size-1)})
+        data = grab_json(url)
+        if "entries" not in data:
+            raise Exception("Missing data in SBS response", data)
+        return data["entries"]
 
     def explode_videos_to_unique_categories(self, all_video_entries):
         for entry_data in all_video_entries: