Skip to content
This repository has been archived by the owner on May 5, 2024. It is now read-only.

建议增加登录/重登录事件 #119

Open
xuefer opened this issue May 6, 2017 · 3 comments
Open

建议增加登录/重登录事件 #119

xuefer opened this issue May 6, 2017 · 3 comments

Comments

@xuefer
Copy link
Contributor

xuefer commented May 6, 2017

SmartQQBot 自己也应该在登录的时候清理一些缓存数据, 看代码似乎是都重新请求服务器获取新的数据了, 并未仔细阅读确认, 不过可以确认部分函数请求服务器并未实行 cache, 插件可能反复调用该函数因此需要实现自行 cache (直到 SmartQQBot 做了 cache)
因此建议增加重登录事件, 插件可以收到事件并做一些数据清理, 特别是 uin 对照表的缓存
增加以下事件通知

  1. Login 登录 (Session Start)
  2. NeedAuth (目前 SmartQQBot 的处理是直接 exit, 建议不要 exit, 要小心比如可能因为安全过敏被服务器拒绝登录需要加以区分, 或者采用每次加大延迟retry的机制, 简单地频繁登录可能会被ban)
  3. Logout (或许该叫 Session End, 毕竟不是主动 log out)
    这样用户可以自行写插件通知拥有者干预操作 (IRC/email/twitter 各种)
    或者插件需要清理数据/重新初始化数据
@winkidney
Copy link
Collaborator

可以以信号的形式增加:)感谢建议
on_logout之类的。

@Yinzo
Copy link
Owner

Yinzo commented May 6, 2017

现在请求的cache都是我零散加在对应的函数里面的,比较混乱,可以考虑使用一个全局的cache管理

@CyrilFeng
Copy link

CyrilFeng commented Oct 17, 2017

[SPAM] 请不要在本项目 Issue 下发布与本 Issue 甚至与本项目无关的内容。

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants