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.

Sign up for GitHub

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

Jump to bottom

增加cookie管理功能 #820

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

增加cookie管理功能 #820

wants to merge 5 commits into from

Conversation

murphysking
Copy link
Contributor

主体功能开发完成，但存在问题：

  • 添加cookie后需要重新对直播间添加监听才能使cookie生效，会导致录制中的直播间中断后重新开始录制，产生两个录制文件；
  • 如果不重新添加监听的话，只有程序重启才能生效.

各位大佬给点建议

@murphysking murphysking changed the title 增加cookie管理功能 WIP 增加cookie管理功能 Dec 25, 2024
@kira1928
Copy link
Collaborator

谢谢 @murphysking ！
今天刚发现这个PR，抱歉！
我给你的 repo 发了一个 PR，里面是我想到的一些修改方法。我们可以在 murphysking#3 讨论~

@kira1928
Copy link
Collaborator

kira1928 commented Feb 1, 2025

大佬试过我的修改了吗？coockie也可以即时生效吗？

@murphysking
Copy link
Contributor Author

murphysking commented Feb 11, 2025 •
edited
Loading

简单测试了下，对录制中的直播间无效，其他可以。目前猜测是程序只在开始录制时获取直播链接，加了cookie不会重新获取链接，只有停止录制重新开始才能生效。

@murphysking murphysking changed the title WIP 增加cookie管理功能 增加cookie管理功能 Feb 22, 2025
@murphysking
Copy link
Contributor Author

所有修改代码都已提交，各位大佬看看

@kira1928
Copy link
Collaborator

非常感谢！你的部分已经做得足够好了，剩下就是等我或原作者确认然后merge和更新版本就好。
更版本好麻烦，我有点没干劲。。orz

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@murphysking @kira1928