Skip to content

fix(video): use raw URLs for video player schemes#453

Open
Lanfei wants to merge 1 commit intoOpenListTeam:mainfrom
Lanfei:fix/video-player-use-raw-url
Open

fix(video): use raw URLs for video player schemes#453
Lanfei wants to merge 1 commit intoOpenListTeam:mainfrom
Lanfei:fix/video-player-use-raw-url

Conversation

@Lanfei
Copy link
Copy Markdown
Contributor

@Lanfei Lanfei commented Mar 28, 2026

Summary / 摘要

  • Use raw URLs instead of download URLs (which involve 302 redirects) for external player schemes, fixing playback failures in players that do not support 302 redirection
  • 外部播放器跳转链接改用原始 URL,而非经过 302 跳转的下载 URL,修复不支持 302 跳转的播放器无法正常播放的问题
  • Affected players / 受影响播放器:IINA、PotPlayer、VLC、Android、nPlayer、OmniPlayer、Fig Player、Infuse、Fileball、MX Player、MX Player Pro、iPlay、mpv

Background / 背景

Some external players (e.g. IINA) do not follow 302 redirects, causing playback to fail when a download URL is used. Switching to raw URLs allows direct access to the resource.

部分外部播放器(如 IINA)不支持 302 跳转,使用下载 URL 时会导致播放失败。改用原始 URL 可直接访问资源,避免此问题。

Test plan / 测试计划

  • Open a video with IINA and confirm it plays correctly / 使用 IINA 打开视频,确认可正常播放
  • Verify other external players (VLC, PotPlayer, etc.) open links correctly / 使用其他外部播放器(VLC、PotPlayer 等)验证跳转正常

外部播放器跳转链接使用原始地址,避免不支持 302 跳转的播放器(如 IINA)无法正常播放。

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
icon: "potplayer",
name: "PotPlayer",
scheme: "potplayer://$durl",
scheme: "potplayer://$url",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PotPlayer实测支持302

icon: "mpv",
name: "mpv",
scheme: "mpv://$edurl",
scheme: "mpv://$eurl",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mpv 实测支持 302

@Lanfei
Copy link
Copy Markdown
Contributor Author

Lanfei commented Apr 1, 2026

@xrgzs 使用原始地址理论上所有播放器都能支持,使用 302 多跳转一层似乎必要性不大,还是说有什么其它层面的考虑?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants