Skip to content

Conversation

@xycld
Copy link
Contributor

@xycld xycld commented Jul 10, 2025

动机

修复我自己遇到的问题:默认下载字幕时,有的时候会遇到ValueError: unknown url type: '/'报错,导致整体被中断

解决方案

增加错误处理,如果字幕文件路径存在问题则不获取,避免中断下载进程

类型

  • ✨ feat: 添加新功能
  • 🐛 fix: 修复 bug
  • 📝 docs: 对文档进行修改
  • ♻️ refactor: 代码重构(既不是新增功能,也不是修改 bug 的代码变动)
  • ⚡ perf: 提高性能的代码修改
  • 🧑‍💻 dx: 优化开发体验
  • 🔨 workflow: 工作流变动
  • 🏷️ types: 类型声明修改
  • 🚧 wip: 工作正在进行中
  • ✅ test: 测试用例添加及修改
  • 🔨 build: 影响构建系统或外部依赖关系的更改
  • 👷 ci: 更改 CI 配置文件和脚本
  • ❓ chore: 其它不涉及源码以及测试的修改
  • ⬆️ deps: 依赖项修改
  • 🔖 release: 发布新版本

@xycld xycld changed the title ✨ feat: 增强字幕获取功能,添加无效URL检查和异常处理 🐛 fix: 增强字幕获取功能,添加无效URL检查和异常处理 Jul 10, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented Jul 10, 2025

CodSpeed Performance Report

Merging #533 will not alter performance

Comparing xycld:main (2c629a9) with main (5d53b6f)

Summary

✅ 30 untouched benchmarks

@SigureMo
Copy link
Member

看起来不是随机问题,麻烦提供下具体的视频链接,滥用 try-except 绝不是最佳实践

@xycld xycld closed this by deleting the head repository Jul 10, 2025
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.

3 participants