feat/videos-only-mode #5

Joren · 2026-05-16T22:55:40+02:00

Joren commented

2026-05-16 22:55:40 +02:00

No description provided.

Joren added 6 commits 2026-05-16 22:55:41 +02:00

add video-only mode 13063c6cc5

fix: normalize List.aspx?folderID= to fragment form for yt-dlp 43392a4132

Canvas stores folder links as List.aspx?folderID=X (query param).
yt-dlp's PanoptoList extractor requires List.aspx#folderID="X"
(fragment with quoted ID) to scope the download to that folder.
Without the fragment form it downloaded the entire Panopto instance
(1806 items instead of 3).

Also drop --no-playlist for list URLs since they are intentional
playlists, and use title/%(title)s.%(ext)s output template for them.

fix: use module_item_redirect OAuth flow for ExternalUrl Panopto items 333e784ce9

The Canvas app authenticates ExternalUrl items via:
  GET session_token?return_to=<module_item_redirect>?display=borderless
  → GET session_url → OAuth2 confirm
  → POST /login/oauth2/accept
  → Panopto Login.aspx?code= → CookieCheck.aspx (sets Panopto cookies)

Our previous code used sessionless_launch (the course-level Panopto tool)
for direct Panopto links, which gave wrong/incomplete Panopto cookies.

Added DownloadExternalPanoptoURL() that replicates the exact app flow.
Falls back to DownloadVideo if no Panopto cookies are obtained.
Both List.aspx (folder playlists) and Viewer.aspx (single videos) are
handled with the correct yt-dlp flags and output templates.

fix: use item.HTMLURL (module_item_redirect) not item.URL for ExternalUrl auth ea9d4dc2dc

For ExternalUrl module items, the Canvas API returns url=null and
html_url=.../module_item_redirect/<id>. We were passing item.URL (empty)
causing the session_token call to return no session_url.

Downloader fixes with videos 522a8b22f8

fix: revert ExternalUrl auth to DownloadVideo, remove dead debug code 2776d057cd

DownloadVideo already handles the Panopto Login.aspx canvas auth form
correctly through its multi-step form submission logic. The real fix for
ExternalUrl Panopto folder links was the URL normalization (List.aspx?folderID=
→ List.aspx#folderID="...") already in DownloadVideo.

Remove DownloadExternalPanoptoURL (dead code) and all debug prints.
Update README with -vo flag documentation.

Joren merged commit 04e7d3e0f4 into main

2026-05-16 22:55:49 +02:00

Joren deleted branch feat/videos-only-mode

2026-05-16 22:55:49 +02:00

Joren referenced this issue from a commit

2026-05-16 22:55:50 +02:00

Merge pull request 'feat/videos-only-mode' (#5) from feat/videos-only-mode into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Joren/CanvasArchiver#5