-
Notifications
You must be signed in to change notification settings - Fork 522
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
Labels
enhancement
New feature or request
Comments
欢迎各位大佬提出修改建议 |
原来如此,好酷的标签! |
小心golang的cookie管理部分代码有点绕,尤其程序启动后添加和更改的cookie不知道能不能顺利生效。 |
很方便的设计 |
目前开发遇到一个问题,添加cookie后需要重新对直播间添加监听才能使cookie生效,会导致录制中的直播间中断后重新开始录制,产生两个录制文件;如果不重新添加监听的话,只有程序重启才能生效。各位有好的办法处理吗? |
你可以发一个wip的pr,然后我们直接针对你的pr代码交流。这样比较方便,可以在代码中交流 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
新功能
基于 #812 (comment) 提出的建议,我规划了如下方案:
1、添加直播间界面增加高级设置,默认折叠,将cookie设置以及后续可能新增的设置项放在里面
2、cookie格式验证仅限于必选项name=value,其他部分没法验证
3、现有监控列表改为多页签显示,增加cookie管理页签,将直播间列表host去重后与cookies合并,可以添加和删除cookie
对于第三点
多页签
方案,目前规划页面形式如下通过页签切换
直播间列表
和Cookie列表
,可以实现Cookie统一管理The text was updated successfully, but these errors were encountered: