[Zoom] Fix extractor

[rafiw]

#32094

Please follow the guide below

• You will be asked some questions, please read them carefully and answer honestly
• Put an x into all the boxes [ ] relevant to your pull request (like that [x])
• Use Preview tab to see how your pull request will actually look like

---

Before submitting a pull request make sure you have:

• Searched the bugtracker for similar pull requests
• Read adding new extractor tutorial
• Read youtube-dl coding conventions and adjusted the code to meet them
• Covered the code with tests (note that PRs without tests will be REJECTED)
• Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

• I am the original author of this code and I am willing to release it under Unlicense
• I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

• Bug fix
• Improvement
• New extractor
• New feature

---

Description of your pull request and other information

This fixes the zoom extractor reported in #32094

[dirkf]

Thanks for your work!

I have some work-in-progress to contribute. We should also ensure align the extractor with yt-dlp's as appropriate.

For now I've just made some initial suggestions.

[dirkf]

I've taken the opportunity of your PR to make a wholesale update of the extractor, and I hope that's OK. It would be a big help if you were able to check that the revised code works well for you.

[rafiw]

Hey, I'm on vacation with very limited access to a computer for the next month.I can take a look when I'm back. In a glance it looks great thanks for the rewrite.

…

On Mon, 8 Jul 2024, 18:33 dirkf, ***@***.***> wrote: I've taken the opportunity of your PR to make a wholesale update of the extractor, and I hope that's OK. It would be a big help if you were able to check that the revised code works well for you. — Reply to this email directly, view it on GitHub <#32828 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAXE4U32565KCWWJKKUASR3ZLJ2G7AVCNFSM6AAAAABKFAMLOCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJTG43DQMJZHE> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[rafiw]

if video doesn't exists like in the test https://us02web.zoom.us/rec/share/hkUk5Zxcga0nkyNGhVCRfzkA2gX_mzgS3LpTxEEWJz9Y_QpIQ4mZFOUx7KZRZDQA.9LGQBdqmDAYgiZ_8 result object will be None and accessing redirectUrl will throw a nasy exception.
maybe something like this?

if download_json['result'] is not None and 'redirectUrl' in download_json['result']:
	redirect_path = download_json['result']['redirectUrl']
	url = urljoin(base_url, redirect_path)
	query['continueMode'] = 'true'
else:
	if 'errorMessage' in download_json:
		raise UnsupportedError(download_json['errorMessage'])
	else:
		raise UnsupportedError("No video found")