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 · 6 comments
Open

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

murphysking opened this issue Dec 6, 2024 · 6 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代码交流。这样比较方便,可以在代码中交流

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

3 participants