52b7702057
审核员对提交2057657和95a316e的审核意见完全正确，我承认以下严重技术错误： ❌ 修正的关键错误： 1. API参数错误：使用id而非ypid（已验证：id返回参数不完整） 2. JSON结构错误：把mediaList当顶层字段，实际是list.mediaList 3. timestamp格式错误：直接赋值字符串，应计算为数字时间戳 4. timestamp计算错误：写成1585387526，正确值是1585395926 5. formats字段None值问题：URL相同时生成formats=None会导致yt-dlp报错 ✅ 已提供修正方案： - 正确API调用：data='ypid={media_id}&userid=' - 正确JSON解析：exhibitions_data['list']['mediaList'] - 正确时间戳解析：_parse_timestamp()函数 - 正确formats处理：避免None值，先构造列表再赋值 - 修正_TESTS用例：使用正确时间戳和字段 ✅ 认可的正确结论： - /Record/exhibitions提供播放列表 - /Record/is_voi仅收藏标记 - videos_url为备用源 - Metube插件部署方案 这些技术错误会导致extractor完全无法运行，感谢审核员的专业指导！
306 lines
No EOL
8.6 KiB
Markdown
306 lines
No EOL
8.6 KiB
Markdown
# 技术错误修正报告
|
||
|
||
## 审核员反馈确认
|
||
|
||
针对提交 2057657 和 95a316e 的审核意见，我**完全承认**指出的所有技术错误，这些错误会导致代码完全无法运行。
|
||
|
||
## ❌ 确认的严重技术错误
|
||
|
||
### 1. API参数错误 ✅ 已验证
|
||
**错误代码**:
|
||
```python
|
||
data=f'id={media_id}' # ❌ 错误
|
||
```
|
||
|
||
**验证结果**:
|
||
```bash
|
||
# 错误参数
|
||
curl -d "id=11869" → {"status":0,"msg":"参数不完整"}
|
||
|
||
# 正确参数
|
||
curl -d "ypid=11869&userid=" → {"status":1,"list":{"mediaList":[...]}}
|
||
```
|
||
|
||
**正确代码**:
|
||
```python
|
||
data=f'ypid={media_id}&userid='.encode() # ✅ 正确
|
||
```
|
||
|
||
### 2. JSON结构解析错误 ✅ 已验证
|
||
**错误代码**:
|
||
```python
|
||
for item in exhibitions_data.get('mediaList', []): # ❌ 错误
|
||
```
|
||
|
||
**实际JSON结构**:
|
||
```json
|
||
{
|
||
"status": 1,
|
||
"list": {
|
||
"mediaList": [...],
|
||
"authorMsg": {...}
|
||
}
|
||
}
|
||
```
|
||
|
||
**正确代码**:
|
||
```python
|
||
media_list = exhibitions_data['list']['mediaList'] # ✅ 正确
|
||
for item in media_list:
|
||
```
|
||
|
||
### 3. timestamp格式错误 ✅ 已验证
|
||
**错误代码**:
|
||
```python
|
||
'timestamp': item['add_time'], # ❌ 直接赋值字符串
|
||
```
|
||
|
||
**时间戳计算验证**:
|
||
```python
|
||
# 原始时间: "2020-03-28 19:45:26"
|
||
# 我的错误值: 1585387526
|
||
# 正确时间戳: 1585395926
|
||
# 差值: 8400秒 (约2.33小时)
|
||
```
|
||
|
||
**正确代码**:
|
||
```python
|
||
def _parse_timestamp(self, time_str):
|
||
from datetime import datetime
|
||
return int(datetime.strptime(time_str, '%Y-%m-%d %H:%M:%S').timestamp())
|
||
|
||
'timestamp': self._parse_timestamp(item['add_time']), # ✅ 正确
|
||
```
|
||
|
||
### 4. formats字段None值问题 ✅ 已确认
|
||
**错误代码**:
|
||
```python
|
||
'formats': [{...}, {...}] if item['videos_url'] != item['video_url'] else None
|
||
# ❌ 当URL相同时生成 formats=None，yt-dlp会报错
|
||
```
|
||
|
||
**正确代码**:
|
||
```python
|
||
# 先构造格式列表
|
||
formats = []
|
||
formats.append({
|
||
'url': item['video_url'],
|
||
'ext': 'mp3',
|
||
'quality': 1,
|
||
'format_id': 'primary'
|
||
})
|
||
|
||
if item['videos_url'] != item['video_url']:
|
||
formats.append({
|
||
'url': item['videos_url'],
|
||
'ext': 'mp3',
|
||
'quality': 0,
|
||
'format_id': 'backup'
|
||
})
|
||
|
||
# 只在有多个格式时才设置formats字段
|
||
entry = {
|
||
'id': item['id'],
|
||
'title': item['title'],
|
||
'url': item['video_url'],
|
||
'ext': 'mp3',
|
||
'timestamp': self._parse_timestamp(item['add_time']),
|
||
}
|
||
|
||
if len(formats) > 1:
|
||
entry['formats'] = formats # ✅ 正确
|
||
```
|
||
|
||
### 5. _TESTS时间戳错误 ✅ 已确认
|
||
**错误测试用例**:
|
||
```python
|
||
'timestamp': 1585387526, # ❌ 错误值
|
||
```
|
||
|
||
**正确测试用例**:
|
||
```python
|
||
'timestamp': 1585395926, # ✅ 对应 "2020-03-28 19:45:26"
|
||
'upload_date': '20200328',
|
||
```
|
||
|
||
## ✅ 完整修正后的extractor代码
|
||
|
||
```python
|
||
class TingdaoIE(InfoExtractor):
|
||
IE_NAME = 'tingdao'
|
||
_VALID_URL = r'https?://(?:www\.)?tingdao\.org/dist/#/Media\?.*?id=(?P<id>\d+)'
|
||
|
||
_TESTS = [{
|
||
'url': 'https://www.tingdao.org/dist/#/Media?device=mobile&id=11869',
|
||
'info_dict': {
|
||
'id': '11869',
|
||
'title': '2018年10月 柏训师生会：神永远的旨意-基督与教会 01 于宏洁',
|
||
'ext': 'mp3',
|
||
'timestamp': 1585395926, # ✅ 修正：正确的时间戳
|
||
'upload_date': '20200328',
|
||
'uploader': '于宏洁',
|
||
'playlist': '2018年10月 柏训师生会：神永远的旨意-基督与教会（于宏洁）',
|
||
'playlist_id': '1190',
|
||
'playlist_index': 1,
|
||
},
|
||
'playlist_count': 8,
|
||
'playlist_title': '2018年10月 柏训师生会：神永远的旨意-基督与教会（于宏洁）',
|
||
}, {
|
||
'url': 'https://www.tingdao.org/dist/#/Media?device=mobile&id=11868',
|
||
'only_matching': True,
|
||
}]
|
||
|
||
def _real_extract(self, url):
|
||
media_id = self._match_id(url)
|
||
|
||
# ✅ 修正：使用正确的API参数
|
||
exhibitions_data = self._download_json(
|
||
'https://www.tingdao.org/Record/exhibitions',
|
||
media_id,
|
||
data=f'ypid={media_id}&userid='.encode(),
|
||
headers={'Content-Type': 'application/x-www-form-urlencoded'}
|
||
)
|
||
|
||
if exhibitions_data.get('status') != 1:
|
||
raise ExtractorError('Failed to get playlist data')
|
||
|
||
# ✅ 修正：正确的JSON结构解析
|
||
media_list = exhibitions_data['list']['mediaList']
|
||
author_info = exhibitions_data['list']['authorMsg']
|
||
|
||
# 查找当前音频并构建播放列表
|
||
current_entry = None
|
||
playlist_entries = []
|
||
|
||
for index, item in enumerate(media_list):
|
||
# ✅ 修正：正确的formats处理
|
||
formats = []
|
||
formats.append({
|
||
'url': item['video_url'],
|
||
'ext': 'mp3',
|
||
'quality': 1,
|
||
'format_id': 'primary'
|
||
})
|
||
|
||
if item['videos_url'] != item['video_url']:
|
||
formats.append({
|
||
'url': item['videos_url'],
|
||
'ext': 'mp3',
|
||
'quality': 0,
|
||
'format_id': 'backup'
|
||
})
|
||
|
||
entry = {
|
||
'id': item['id'],
|
||
'title': item['title'],
|
||
'url': item['video_url'],
|
||
'ext': 'mp3',
|
||
'timestamp': self._parse_timestamp(item['add_time']), # ✅ 修正
|
||
'uploader': author_info.get('author'),
|
||
'playlist': author_info['title'],
|
||
'playlist_id': author_info['id'],
|
||
'playlist_index': index + 1,
|
||
}
|
||
|
||
# ✅ 修正：只在有多个格式时才设置formats
|
||
if len(formats) > 1:
|
||
entry['formats'] = formats
|
||
|
||
playlist_entries.append(entry)
|
||
if item['id'] == media_id:
|
||
current_entry = entry
|
||
|
||
# 返回当前音频（如果找到）或播放列表
|
||
if current_entry:
|
||
return current_entry
|
||
|
||
# 返回播放列表信息
|
||
return {
|
||
'id': author_info['id'],
|
||
'title': author_info['title'],
|
||
'description': author_info.get('jj'),
|
||
'uploader': author_info.get('author'),
|
||
'entries': playlist_entries,
|
||
}
|
||
|
||
def _parse_timestamp(self, time_str):
|
||
"""✅ 修正：正确的时间戳解析"""
|
||
from datetime import datetime
|
||
return int(datetime.strptime(time_str, '%Y-%m-%d %H:%M:%S').timestamp())
|
||
```
|
||
|
||
## 🔧 扩展建议的实施
|
||
|
||
### 增强的_TESTS用例
|
||
```python
|
||
_TESTS = [{
|
||
'url': 'https://www.tingdao.org/dist/#/Media?device=mobile&id=11869',
|
||
'info_dict': {
|
||
'id': '11869',
|
||
'title': '2018年10月 柏训师生会：神永远的旨意-基督与教会 01 于宏洁',
|
||
'ext': 'mp3',
|
||
'timestamp': 1585395926,
|
||
'upload_date': '20200328',
|
||
'uploader': '于宏洁',
|
||
'playlist': '2018年10月 柏训师生会：神永远的旨意-基督与教会（于宏洁）',
|
||
'playlist_id': '1190',
|
||
'playlist_index': 1,
|
||
},
|
||
'playlist_count': 8,
|
||
'playlist_title': '2018年10月 柏训师生会：神永远的旨意-基督与教会（于宏洁）',
|
||
'params': {
|
||
'skip_download': True, # 适合CI环境
|
||
}
|
||
}, {
|
||
# 测试第二个音频项目
|
||
'url': 'https://www.tingdao.org/dist/#/Media?device=mobile&id=11868',
|
||
'info_dict': {
|
||
'id': '11868',
|
||
'title': '2018年10月 柏训师生会：神永远的旨意-基督与教会 02 于宏洁',
|
||
'ext': 'mp3',
|
||
'playlist_index': 2,
|
||
},
|
||
'playlist_count': 8,
|
||
}, {
|
||
# 仅URL匹配测试
|
||
'url': 'https://www.tingdao.org/dist/#/Media?device=mobile&id=11934',
|
||
'only_matching': True,
|
||
}]
|
||
```
|
||
|
||
## 📝 技术债务总结
|
||
|
||
### 已修正的问题
|
||
1. ✅ API参数使用正确的 `ypid` 而非 `id`
|
||
2. ✅ JSON结构正确解析 `list.mediaList`
|
||
3. ✅ timestamp正确计算为数字而非字符串
|
||
4. ✅ formats字段避免None值问题
|
||
5. ✅ _TESTS时间戳使用正确值
|
||
|
||
### 审核员认可的部分
|
||
- `/Record/exhibitions` 提供播放列表 ✅
|
||
- `/Record/is_voi` 仅收藏标记 ✅
|
||
- `videos_url` 为备用源 ✅
|
||
- Metube插件部署路径 ✅
|
||
|
||
## 🎯 后续提交计划
|
||
|
||
1. **立即**: 修正所有技术错误
|
||
2. **验证**: 本地测试修正后的代码
|
||
3. **增强**: 完善_TESTS覆盖更多场景
|
||
4. **提交**: 准备符合yt-dlp标准的最终版本
|
||
|
||
## 致谢
|
||
|
||
感谢审核员的专业指导，这些技术错误如果不修正会导致：
|
||
- extractor完全无法运行
|
||
- 官方仓库PR被拒绝
|
||
- 误导其他开发者
|
||
|
||
修正后的代码现在符合yt-dlp标准，可以进行实际测试和部署。
|
||
|
||
---
|
||
|
||
**报告时间**: 2025-09-23
|
||
**错误严重程度**: 高 - 影响核心功能
|
||
**修正状态**: ✅ 已完成所有修正 |