Repository: yt-dlp/yt-dlp
File: yt_dlp/extractor/allstar.py (Python)

Code before:
    return urljoin('https://media.allstar.gg/', str_or_none(path))


def _profile_url_or_none(path):
    return urljoin('https:/allstar.gg/u/', str_or_none(path))


def _parse_video_data(video_data):
    return traverse_obj(video_data, {
        'id': ('_id', {str_or_none}),
        'display_id': ('shareId', {str_or_none}),
        'title': ('clipTitle', {str_or_none}),
        'url': ('clipLink', {_media_url_or_none}),
        'thumbnail': ('clipImageThumb', {_media_url_or_none}),
        'duration': ('clipLength', {int_or_none}),
        'filesize': ('clipSizeBytes', {int_or_none}),
        'timestamp': ('createdDate', {int_or_none}),
        'uploader': ('username', {str_or_none}),
        'uploader_id': ('user', '_id', {str_or_none}),
        'uploader_url': ('user', '_id', {_profile_url_or_none}),
        'view_count': ('views', {int_or_none}),
        'categories': ('game', {str_or_none}),
    })


def _set_webpage_url(info_dict):
    video_id = info_dict.get('id')
    video_url = info_dict.get('url')

    if video_url is None or video_id is None:
        return info_dict

    base_name = 'clip' if '/clips/' in video_url else 'montage'
    info_dict['webpage_url'] = f'https://allstar.gg/{base_name}?{base_name}={video_id}'
    info_dict['webpage_url_basename'] = base_name

    return info_dict


class AllstarBase(InfoExtractor):
    def _send_query(self, query, variables={}, path=(), video_id=None):
        response = self._download_json(
            'https://a1.allstar.gg/graphql', video_id,
            headers={'content-type': 'application/json'},
            data=json.dumps({'variables': variables, 'query': query}).encode())

        errors = response.get('errors')
        if errors:
            raise ExtractorError(errors, expected=True)

        return traverse_obj(response, path)

Review comment (suggestion):
Maybe this should become part of `_parse_video_data`? i.e., always setting `webpage_url`.
Would like to hear what others think regarding this one.

Code after:

        info = traverse_obj(video_data, {
            'id': ('_id', {str}),
            'display_id': ('shareId', {str}),
            'title': ('clipTitle', {str}),
            'url': ('clipLink', {media_url_or_none}),
            'thumbnails': (('clipImageThumb', 'clipImageSource'), {'url': {media_url_or_none}}),
            'duration': ('clipLength', {int_or_none}),
            'filesize': ('clipSizeBytes', {int_or_none}),
            'timestamp': ('createdDate', {functools.partial(int_or_none, scale=1000)}),
            'uploader': ('username', {str}),
            'uploader_id': ('user', '_id', {str}),
            'view_count': ('views', {int_or_none}),
        })

        if info.get('id') and info.get('url'):
            basename = 'clip' if '/clips/' in info['url'] else 'montage'
            info['webpage_url'] = f'https://allstar.gg/{basename}?{basename}={info["id"]}'

        info.update({
            'extractor_key': AllstarIE.ie_key(),
            'extractor': AllstarIE.IE_NAME,
            'uploader_url': urljoin('https://allstar.gg/u/', info.get('uploader_id')),
        })

        return info

    def _call_api(self, query, variables, path, video_id=None, note=None):
        response = self._download_json(
            'https://a1.allstar.gg/graphql', video_id, note=note,
            headers={'content-type': 'application/json'},
            data=json.dumps({'variables': variables, 'query': query}).encode())

        errors = traverse_obj(response, ('errors', ..., 'message', {str}))
        if errors:
            raise ExtractorError('; '.join(errors))

        return traverse_obj(response, path)


class AllstarIE(AllstarBaseIE):
    _VALID_URL = r'https?://(?:www\.)?allstar\.gg/(?P<type>(?:clip|montage))\?(?P=type)=(?P<id>[^/?#&]+)'

    _TESTS = [{
        'url': 'https://allstar.gg/clip?clip=64482c2da9eec30008a67d1b',
        'info_dict': {
            'id': '64482c2da9eec30008a67d1b',
            'title': '4K on Inferno',
            'url': 'md5:66befb5381eef0c9456026386c25fa55',
            'thumbnail': r're:https://media\.allstar\.gg/.+\.(?:png|jpg)$',
            'uploader': 'chrk.',