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

Code before:

    _TESTS = [{
        'url': 'https://demosubdomain.webex.com/demositeurl/ldr.php?RCID=e58e803bc0f766bb5f6376d2e86adb5b',
        'only_matching': True,
    }, {
        'url': 'http://demosubdomain.webex.com/demositeurl/lsr.php?RCID=bc04b4a7b5ea2cc3a493d5ae6aaff5d7',
        'only_matching': True,
    }, {
        'url': 'https://demosubdomain.webex.com/recordingservice/sites/demositeurl/recording/88e7a42f7b19f5b423c54754aecc2ce9/playback',
        'only_matching': True,
    }]

    def _real_extract(self, url):
        mobj = re.match(self._VALID_URL, url)
        rcid = mobj.group('rcid')
        if rcid is not None:
            # Visit URL and extract redirection URL from inline JavaScript
            webpage = self._download_webpage(url, None, note='Getting video ID')
            mobj = re.search(self._VALID_URL, webpage)
            if mobj is None:
                raise ExtractorError('Could not get video ID; visit the URL manually and use the redirection URL')
            url = mobj.group()
            return {
                '_type': 'url',
                'url': url
            }

        # Sometimes the siteurl is an alias, we get the real one via HTTP
        # redirection (method HEAD does not work here)
        url = self._request_webpage(
            url, None, note='Resolving final URL',
            errnote='Could not resolve final URL').geturl()
        mobj = re.match(self._VALID_URL, url)
        subdomain = mobj.group('subdomain')
        siteurl = mobj.group('site_1') or mobj.group('site_2')
        video_id = mobj.group('id_1') or mobj.group('id_2') or mobj.group('id_3')

        try:
            stream = self._download_json(
                'https://%s.webex.com/webappng/api/v1/recordings/%s/stream' % (subdomain, video_id),
                video_id, query={
                    'siteurl': siteurl
                })
        except ExtractorError:
            self.raise_login_required()

        video_id = stream.get('recordUUID') or video_id

        formats = [{
            'format_id': 'video',
            'url': stream['fallbackPlaySrc'],

Review comment (refactor):
```suggestion
            return self.url_result(url)
```

OR

I think it is even better to just set the new `mobj` and `rcid` and continue. Since the same extractor is guarenteed to catch the url back, there is no need to go through the handover process

Code after:

    def _real_extract(self, url):
        mobj = self._match_valid_url(url)
        rcid = mobj.group('rcid')
        if rcid:
            webpage = self._download_webpage(url, None, note='Getting video ID')
            url = self._search_regex(self._VALID_URL, webpage, 'redirection url', group='url')
        url = self._request_webpage(url, None, note='Resolving final URL').geturl()
        mobj = self._match_valid_url(url)
        subdomain = mobj.group('subdomain')
        siteurl = mobj.group('siteurl_1') or mobj.group('siteurl_2')
        video_id = mobj.group('id')

        stream = self._download_json(
            'https://%s.webex.com/webappng/api/v1/recordings/%s/stream' % (subdomain, video_id),
            video_id, fatal=False, query={'siteurl': siteurl})
        if not stream:
            self.raise_login_required(method='cookies')

        video_id = stream.get('recordUUID') or video_id

        formats = [{
            'format_id': 'video',
            'url': stream['fallbackPlaySrc'],
            'ext': 'mp4',
            'vcodec': 'avc1.640028',
            'acodec': 'mp4a.40.2',
        }]
        if stream.get('preventDownload') is False:
            mp4url = try_get(stream, lambda x: x['downloadRecordingInfo']['downloadInfo']['mp4URL'])
            if mp4url:
                formats.append({
                    'format_id': 'video',
                    'url': mp4url,
                    'ext': 'mp4',
                    'vcodec': 'avc1.640028',
                    'acodec': 'mp4a.40.2',
                })
            audiourl = try_get(stream, lambda x: x['downloadRecordingInfo']['downloadInfo']['audioURL'])
            if audiourl:
                formats.append({
                    'format_id': 'audio',
                    'url': audiourl,
                    'ext': 'mp3',
                    'vcodec': 'none',
                    'acodec': 'mp3',
                })
        self._sort_formats(formats)

        return {
            'id': video_id,