diff options
| author | Alexandre Flament <alex@al-f.net> | 2019-04-09 03:26:27 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-04-09 03:26:27 +0200 |
| commit | ca09d9102310e24c08db4c413e695fabfd5def4f (patch) | |
| tree | 8eab37042642564a63a2be8f11f3c85ec42fd6fa /searx | |
| parent | 3ee804ec8c64e65ea180b2da7a7d79feb73bee5f (diff) | |
| parent | e868650df7164ad36079c1525f8efd0d5c56fe08 (diff) | |
Merge pull request #1544 from MarcAbonce/youtube_fix
[fix] get YouTube results
Diffstat (limited to 'searx')
| -rw-r--r-- | searx/engines/youtube_noapi.py | 70 |
1 files changed, 34 insertions, 36 deletions
diff --git a/searx/engines/youtube_noapi.py b/searx/engines/youtube_noapi.py index 9f01841f6..3bf25932b 100644 --- a/searx/engines/youtube_noapi.py +++ b/searx/engines/youtube_noapi.py @@ -8,7 +8,8 @@ # @stable no # @parse url, title, content, publishedDate, thumbnail, embedded -from lxml import html +from functools import reduce +from json import loads from searx.engines.xpath import extract_text from searx.utils import list_get from searx.url_utils import quote_plus @@ -34,20 +35,6 @@ embedded_url = '<iframe width="540" height="304" ' +\ base_youtube_url = 'https://www.youtube.com/watch?v=' -# specific xpath variables -results_xpath = "//ol/li/div[contains(@class, 'yt-lockup yt-lockup-tile yt-lockup-video vve-check')]" -url_xpath = './/h3/a/@href' -title_xpath = './/div[@class="yt-lockup-content"]/h3/a' -content_xpath = './/div[@class="yt-lockup-content"]/div[@class="yt-lockup-description yt-ui-ellipsis yt-ui-ellipsis-2"]' - - -# returns extract_text on the first result selected by the xpath or None -def extract_text_from_dom(result, xpath): - r = result.xpath(xpath) - if len(r) > 0: - return extract_text(r[0]) - return None - # do search-request def request(query, params): @@ -63,27 +50,38 @@ def request(query, params): def response(resp): results = [] - dom = html.fromstring(resp.text) - - # parse results - for result in dom.xpath(results_xpath): - videoid = list_get(result.xpath('@data-context-item-id'), 0) - if videoid is not None: - url = base_youtube_url + videoid - thumbnail = 'https://i.ytimg.com/vi/' + videoid + '/hqdefault.jpg' - - title = extract_text_from_dom(result, title_xpath) or videoid - content = extract_text_from_dom(result, content_xpath) - - embedded = embedded_url.format(videoid=videoid) - - # append result - results.append({'url': url, - 'title': title, - 'content': content, - 'template': 'videos.html', - 'embedded': embedded, - 'thumbnail': thumbnail}) + results_data = resp.text[resp.text.find('ytInitialData'):] + results_data = results_data[results_data.find('{'):results_data.find(';\n')] + + results_json = loads(results_data) if results_data else {} + sections = results_json.get('contents', {})\ + .get('twoColumnSearchResultsRenderer', {})\ + .get('primaryContents', {})\ + .get('sectionListRenderer', {})\ + .get('contents', []) + + for section in sections: + for video_container in section.get('itemSectionRenderer', {}).get('contents', []): + video = video_container.get('videoRenderer', {}) + videoid = video.get('videoId') + if videoid is not None: + url = base_youtube_url + videoid + thumbnail = 'https://i.ytimg.com/vi/' + videoid + '/hqdefault.jpg' + title = video.get('title', {}).get('simpleText', videoid) + description_snippet = video.get('descriptionSnippet', {}) + if 'runs' in description_snippet: + content = reduce(lambda a, b: a + b.get('text', ''), description_snippet.get('runs'), '') + else: + content = description_snippet.get('simpleText', '') + embedded = embedded_url.format(videoid=videoid) + + # append result + results.append({'url': url, + 'title': title, + 'content': content, + 'template': 'videos.html', + 'embedded': embedded, + 'thumbnail': thumbnail}) # return results return results |