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

Add ixigua extractor #1049

Merged
merged 19 commits into from
Mar 4, 2022
Merged

Add ixigua extractor #1049

merged 19 commits into from
Mar 4, 2022

Conversation

dfang
Copy link
Contributor

@dfang dfang commented Mar 3, 2022

close #846
close #654

@dfang
Copy link
Contributor Author

dfang commented Mar 3, 2022

从url https://www.ixigua.com/6926875601139991053 中获取到item_id: 6926875601139991053, 然后拼接https://www.ixigua.com/api/public/videov2/brief/details?group_id=7067848979140051456 可以解析到video_list, 此方法解析到的视频都有水印

app/register.go Outdated Show resolved Hide resolved
extractors/ixigua/ixigua.go Outdated Show resolved Hide resolved
utils/utils.go Outdated Show resolved Hide resolved
extractors/ixigua/ixigua.go Outdated Show resolved Hide resolved
extractors/ixigua/ixigua.go Outdated Show resolved Hide resolved
extractors/ixigua/ixigua.go Outdated Show resolved Hide resolved
extractors/ixigua/ixigua.go Outdated Show resolved Hide resolved
extractors/ixigua/ixigua.go Outdated Show resolved Hide resolved
extractors/ixigua/ixigua.go Outdated Show resolved Hide resolved
@dfang
Copy link
Contributor Author

dfang commented Mar 4, 2022

迟点再merge & close,目前只支持https://www.ixigua.com/xxxx 这种格式, https://v.ixigua.com/xxxx 还需要完善下

支持ixigua两种格式的url
@dfang
Copy link
Contributor Author

dfang commented Mar 4, 2022

可以了, 这种格式的url https://m.toutiao.com/is/dtj1pND/ 也支持了

@dfang
Copy link
Contributor Author

dfang commented Mar 4, 2022

close #654

extractors/ixigua/ixigua.go Show resolved Hide resolved
extractors/ixigua/ixigua.go Outdated Show resolved Hide resolved
extractors/ixigua/ixigua.go Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
extractors/ixigua/ixigua.go Outdated Show resolved Hide resolved
extractors/ixigua/ixigua_test.go Outdated Show resolved Hide resolved
extractors/kuaishou/kuaishou.go Outdated Show resolved Hide resolved
Co-authored-by: Xinzhao Xu <z2d@jifangcheng.com>
dfang and others added 2 commits March 4, 2022 17:13
Co-authored-by: Xinzhao Xu <z2d@jifangcheng.com>
Co-authored-by: Xinzhao Xu <z2d@jifangcheng.com>
@dfang
Copy link
Contributor Author

dfang commented Mar 4, 2022

数据返回应该用哪种格式

1
2

@iawia002
Copy link
Owner

iawia002 commented Mar 4, 2022

数据返回应该用哪种格式

1 2

我们只有一种格式

@dfang
Copy link
Contributor Author

dfang commented Mar 4, 2022

数据返回应该用哪种格式
1 2

我们只有一种格式

1 还是2

@iawia002
Copy link
Owner

iawia002 commented Mar 4, 2022

1,你 2 是咋搞出来的,多个是针对不同 URL 的,同一个视频只有一个

@dfang
Copy link
Contributor Author

dfang commented Mar 4, 2022

1,你 2 是咋搞出来的,多个是针对不同 URL 的,同一个视频只有一个

2 是编的

OK,多个不同的quality 还是同属于一个stream的

为什么extractor的返回值是数组([]*extractors.Data) ?

@iawia002
Copy link
Owner

iawia002 commented Mar 4, 2022

为什么extractor的返回值是数组([]*extractors.Data) ?

因为有些网站是支持 playlist 下载的

@dfang
Copy link
Contributor Author

dfang commented Mar 4, 2022

为什么extractor的返回值是数组([]*extractors.Data) ?

因为有些网站是支持 playlist 下载的

好的 理解了

@dfang
Copy link
Contributor Author

dfang commented Mar 4, 2022

返回的stream里的 parts 需要按照清晰度排序吗? 需要设置一个default quality的吗?

有哪个extractor 的 parts 是返回多个的,可以参考?

@iawia002
Copy link
Owner

iawia002 commented Mar 4, 2022

返回的stream里的 parts 需要按照清晰度排序吗? 需要设置一个default quality的吗?

这个你不用管,这是统一做的

有哪个extractor 的 parts 是返回多个的,可以参考?

多个 parts 是同一个视频有多段需要合并的情况,你没有就不用管

README.md Outdated Show resolved Hide resolved
extractors/ixigua/ixigua.go Outdated Show resolved Hide resolved
extractors/ixigua/ixigua.go Outdated Show resolved Hide resolved
extractors/ixigua/ixigua.go Outdated Show resolved Hide resolved
extractors/ixigua/ixigua_test.go Outdated Show resolved Hide resolved
test/utils.go Show resolved Hide resolved
Copy link
Owner

@iawia002 iawia002 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@iawia002 iawia002 merged commit d15e198 into iawia002:master Mar 4, 2022
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.

support request for "ixigua.com"
3 participants