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

Fix youtube source request #484

Closed
wants to merge 2 commits into from
Closed

Conversation

xianingz
Copy link

我改的几点:

  1. 我把find.js里面的https改成了http,这个可能因为我个人网络block了https,可能大部分不需要。
  2. signature函数中修改了关于抓取最新的youtube解码函数的代码
  3. signature函数中id=null时request会报错,不知道是什么原因,我就给设置为默认的id。

@nondanee
Copy link
Owner

😂 发现了,重点就是因为 null,默认 id 没设置成,其他的都不用改的

return stream.url || (target.sp.includes('sig') ? cache(signature, null, 24 * 60 * 60 * 1000).then(sign => target.url + '&sig=' + sign(target.s)) : target.url)

我不 merge 了啊

@nondanee nondanee closed this Apr 10, 2020
Rasta0501 added a commit to Rasta0501/UnblockNeteaseMusic that referenced this pull request Apr 17, 2020
fix youtube sig extraction error (miss default id) nondanee#481 nondanee#484
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