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

BUG: 海外机器无法下载国内平台的歌曲 #10

Closed
docase opened this issue Sep 9, 2022 · 3 comments · Fixed by #12
Closed

BUG: 海外机器无法下载国内平台的歌曲 #10

docase opened this issue Sep 9, 2022 · 3 comments · Fixed by #12

Comments

@docase
Copy link

docase commented Sep 9, 2022

酷我

./media-get -u "https://www.kuwo.cn/play_detail/236967605" - t audio

panic: invalid character 'æ' looking for beginning of value

goroutine 1 [running]:
main.main()
./main.go:24 +0xab

QQ音乐

.\media-get -u "http://y.qq.com/n/ryqq/songDetail/0021bATQ1TadVT" - t audio

panic: can not get song url

goroutine 1 [running]:
main.main()
./main.go:24 +0xab

抖音 bilibili正常

@foamzou
Copy link
Owner

foamzou commented Sep 9, 2022

国内平台大多只允许国内 IP 访问。你的大概率是海外机器?

参考这个 PR,计划添加 proxy。如果有国内代理的话,到时候 feature launch 之后可以配置
#8 (comment)

@foamzou foamzou changed the title BUG: 无法下载歌曲 BUG: 海外机器无法下载国内平台的歌曲 Sep 19, 2022
@mar-heaven
Copy link
Contributor

mar-heaven commented Sep 22, 2022

国内平台大多只允许国内 IP 访问。你的大概率是海外机器?

参考这个 PR,计划添加 proxy。如果有国内代理的话,到时候 feature launch 之后可以配置 #8 (comment)

周末出去浪了,feature放国庆实现。

@foamzou
Copy link
Owner

foamzou commented Sep 24, 2022

国内平台大多只允许国内 IP 访问。你的大概率是海外机器?
参考这个 PR,计划添加 proxy。如果有国内代理的话,到时候 feature launch 之后可以配置 #8 (comment)

周末出去浪了,feature放国庆实现。

Thx bro, just take your time.

chuan-T pushed a commit to chuan-T/media-get that referenced this issue Jan 17, 2023
foamzou pushed a commit that referenced this issue Jan 17, 2023
* feat: support proxy setting

fix #10

* fix: unit test out of range panic

* perf: compatible files have no errors

* perf: code optimize

Co-authored-by: Liu <liu.chuan@siemens.com>
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 a pull request may close this issue.

3 participants