-
Notifications
You must be signed in to change notification settings - Fork 13
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
关于权限功能具体实现的问题 #110
Comments
在权限API设计的早期我已经提及请求问题,我们现在的问题是没有更简单的形如 isPermissionSet(User who, KOOKPermission what) 的 HTTP API 接口,导致我们想要计算权限值只能请求所有用户拥有的角色,然后逐一判断。 |
已优化计算权限时产生大量的http请求. |
等待主要维护者 @RealSeek @xiaoACE6716 回复。我没有任何意见。 |
目前已有功能已符合我的预期,蹲一个持久化 |
我认为目前得PermissionContext还是太多余了,应该取消并不再支持直接判断built-in权限,而是作为可选参数(重载方法)支持,并缓存权限结果一定时间后清除缓存 |
此 issue 交由 @huanmeng-qwq 全权处理,我仅在此 issue 有完整处理方案时 review. |
尝试使用了一下新功能,基本符合我的预期要求,不过还是存在一些问题。
https://pastebin.ubuntu.com/p/WK2wD38JQQ/
首先从这份debug.log中可以看到,有许多重复的网络请求,我想他们的结果应该是一样的,所以是否只需要请求第一次就完成了?
然后就是自定义权限没有持久化
这里添加完权限后重启机器人就丢失了
The text was updated successfully, but these errors were encountered: