Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ytdl_hook.lua parse playlist error on bilibili interactive video #15976

Open
6 tasks done
verygoodlee opened this issue Feb 27, 2025 · 1 comment
Open
6 tasks done

ytdl_hook.lua parse playlist error on bilibili interactive video #15976

verygoodlee opened this issue Feb 27, 2025 · 1 comment
Labels

Comments

@verygoodlee
Copy link
Contributor

verygoodlee commented Feb 27, 2025

mpv Information

mpv v0.39.0-927-g5338f4ba7 Copyright © 2000-2025 mpv/MPlayer/mplayer2 projects
 built on Feb 26 2025 16:05:41
libplacebo version: v7.350.0 (v7.349.0-37-g7de2415-dirty)
FFmpeg version: N-118547-ge24920375
FFmpeg library versions:
   libavcodec      61.33.102
   libavdevice     61.4.100
   libavfilter     10.9.100
   libavformat     61.9.107
   libavutil       59.57.100
   libswresample   5.4.100
   libswscale      8.13.100

Other Information

- Windows version: Win11 24H2
- GPU model, driver and version:
- Source of mpv: https://github.com/zhongfly/mpv-winbuild/releases/tag/2025-02-26-5338f4b
- Latest known working version:
- Issue started after the following happened:

Reproduction Steps

Interactive video is a special type of video in bilibili, different choices will have different results, yt-dlp outputs all branches as a playlist.
https://www.bilibili.com/video/BV1g5pqeBEXP/

Expected Behavior

N/A

Actual Behavior

Image
Line 1167 site is nil

mpv/player/lua/ytdl_hook.lua

Lines 1167 to 1175 in 5338f4b

if not site:find("://") then
-- youtube extractor provides only IDs,
-- others come prefixed with the extractor name and ":"
local prefix = site:find(":") and "ytdl://" or
"https://youtu.be/"
playlist_url = prefix .. site
elseif url_is_safe(site) then
playlist_url = site
end

Log File

yt-dlp_stdout.json

Sample Files

N/A

I carefully read all instruction and confirm that I did the following:

  • I tested with the latest mpv version to validate that the issue is not already fixed.
  • I provided all required information including system and mpv version.
  • I produced the log file with the exact same set of files, parameters, and conditions used in "Reproduction Steps", with the addition of --log-file=output.txt.
  • I produced the log file while the behaviors described in "Actual Behavior" were actively observed.
  • I attached the full, untruncated log file.
  • I attached the backtrace in the case of a crash.
@kasper93 kasper93 modified the milestone: Release v0.40.0 Feb 28, 2025
@kasper93
Copy link
Contributor

kasper93 commented Mar 4, 2025

I've taken a look with hope for small bugfix, but this kind of playlist is not supported by ytdl_hook.lua. We only support lists of urls or multi-segment m3u8. The video in this issue is returning playlist with all chunks as separate extracted entities and there is currently no way to support that without refactoring the parser.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants