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

关于新增cookie管理功能的讨论 #814

Open
murphysking opened this issue Dec 6, 2024 · 8 comments
Open

关于新增cookie管理功能的讨论 #814

murphysking opened this issue Dec 6, 2024 · 8 comments
Labels
enhancement New feature or request

Comments

@murphysking
Copy link
Contributor

murphysking commented Dec 6, 2024

新功能

基于 #812 (comment) 提出的建议,我规划了如下方案:
1、添加直播间界面增加高级设置,默认折叠,将cookie设置以及后续可能新增的设置项放在里面
2、cookie格式验证仅限于必选项name=value,其他部分没法验证
3、现有监控列表改为多页签显示,增加cookie管理页签,将直播间列表host去重后与cookies合并,可以添加和删除cookie
对于第三点多页签方案,目前规划页面形式如下
image
image
通过页签切换直播间列表Cookie列表,可以实现Cookie统一管理

@murphysking murphysking added the enhancement New feature or request label Dec 6, 2024
@murphysking
Copy link
Contributor Author

欢迎各位大佬提出修改建议

@kira1928
Copy link
Collaborator

kira1928 commented Dec 6, 2024

原来如此,好酷的标签!
支持!

@kira1928
Copy link
Collaborator

kira1928 commented Dec 6, 2024

小心golang的cookie管理部分代码有点绕,尤其程序启动后添加和更改的cookie不知道能不能顺利生效。
程序实现后可以重点测试一下这个方面。

@124959377
Copy link

很方便的设计

@murphysking
Copy link
Contributor Author

目前开发遇到一个问题,添加cookie后需要重新对直播间添加监听才能使cookie生效,会导致录制中的直播间中断后重新开始录制,产生两个录制文件;如果不重新添加监听的话,只有程序重启才能生效。各位有好的办法处理吗?

@kira1928
Copy link
Collaborator

你可以发一个wip的pr,然后我们直接针对你的pr代码交流。这样比较方便,可以在代码中交流

@liyunhegh
Copy link

liyunhegh commented Dec 24, 2024

目前开发遇到一个问题,添加cookie后需要重新对直播间添加监听才能使cookie生效,会导致录制中的直播间中断后重新开始录制,产生两个录制文件;如果不重新添加监听的话,只有程序重启才能生效。各位有好的办法处理吗?

应用新cookie时,如果有正在进行的录制,在后台开一个新的带cookie的录制进程,和原来不带cookie的并行运行一段时间,然后关掉老的录制进程

  1. 并行运行防止出现连接时丢失画面问题
  2. 可以做到对用户无感,顶多是多一个视频分段,这个应该是可接受的

@murphysking
Copy link
Contributor Author

我试试。最近比较忙,过年前估计完不成了

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants