修复权限问题 #1225
Open
修复权限问题 #1225
+18
−1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
## Sourcery 评审者指南
此拉取请求通过添加管理员权限要求来限制对多个命令的访问。这可以防止权限较低的用户禁用默认插件并导致其他问题。
_由于更改看起来很简单，不需要可视化表示，因此未生成图表。_
### 文件级别更改
| Change | Details | Files |
| ------ | ------- | ----- |
| 为多个命令添加了管理员权限要求。 | <ul><li>向 `llm` 命令添加了 `@filter.permission_type(filter.PermissionType.ADMIN)` 装饰器。</li><li>向 `plugin` 命令添加了 `@filter.permission_type(filter.PermissionType.ADMIN)` 装饰器。</li><li>向 `t2i` 命令添加了 `@filter.permission_type(filter.PermissionType.ADMIN)` 装饰器。</li><li>向 `tts` 命令添加了 `@filter.permission_type(filter.PermissionType.ADMIN)` 装饰器。</li><li>向 `sid` 命令添加了 `@filter.permission_type(filter.PermissionType.ADMIN)` 装饰器。</li><li>向 `provider` 命令添加了 `@filter.permission_type(filter.PermissionType.ADMIN)` 装饰器。</li><li>向 `model` 命令添加了 `@filter.permission_type(filter.PermissionType.ADMIN)` 装饰器。</li><li>向 `history` 命令添加了 `@filter.permission_type(filter.PermissionType.ADMIN)` 装饰器。</li><li>向 `ls` 命令添加了 `@filter.permission_type(filter.PermissionType.ADMIN)` 装饰器。</li><li>向 `switch` 命令添加了 `@filter.permission_type(filter.PermissionType.ADMIN)` 装饰器。</li><li>向 `rename` 命令添加了 `@filter.permission_type(filter.PermissionType.ADMIN)` 装饰器。</li><li>向 `del` 命令添加了 `@filter.permission_type(filter.PermissionType.ADMIN)` 装饰器。</li><li>向 `set` 命令添加了 `@filter.permission_type(filter.PermissionType.ADMIN)` 装饰器。</li><li>向 `unset` 命令添加了 `@filter.permission_type(filter.PermissionType.ADMIN)` 装饰器。</li><li>向 `gewe_logout` 命令添加了 `@filter.permission_type(filter.PermissionType.ADMIN)` 装饰器。</li><li>向 `gewe_code` 命令添加了 `@filter.permission_type(filter.PermissionType.ADMIN)` 装饰器。</li></ul> | `packages/astrbot/main.py` |
---
<details>
<summary>提示和命令</summary>
#### 与 Sourcery 互动
- **触发新的审查：** 在拉取请求上评论 `@sourcery-ai review`。
- **继续讨论：** 直接回复 Sourcery 的审查评论。
- **从审查评论生成 GitHub issue：** 通过回复审查评论，要求 Sourcery 从审查评论创建一个 issue。您也可以回复审查评论并使用 `@sourcery-ai issue` 从中创建一个 issue。
- **生成拉取请求标题：** 在拉取请求标题中的任何位置写入 `@sourcery-ai` 以随时生成标题。您也可以在拉取请求上评论 `@sourcery-ai title` 以随时（重新）生成标题。
- **生成拉取请求摘要：** 在拉取请求正文中的任何位置写入 `@sourcery-ai summary` 以随时在您想要的位置生成 PR 摘要。您也可以在拉取请求上评论 `@sourcery-ai summary` 以随时（重新）生成摘要。
- **生成评审者指南：** 在拉取请求上评论 `@sourcery-ai guide` 以随时（重新）生成评审者指南。
- **解决所有 Sourcery 评论：** 在拉取请求上评论 `@sourcery-ai resolve` 以解决所有 Sourcery 评论。如果您已经处理了所有评论并且不想再看到它们，这将非常有用。
- **驳回所有 Sourcery 审查：** 在拉取请求上评论 `@sourcery-ai dismiss` 以驳回所有现有的 Sourcery 审查。如果您想以新的审查重新开始，这将特别有用 - 不要忘记评论 `@sourcery-ai review` 以触发新的审查！
- **为 issue 生成行动计划：** 在 issue 上评论 `@sourcery-ai plan` 以生成行动计划。
#### 自定义您的体验
访问您的 [仪表板](https://app.sourcery.ai) 以：
- 启用或禁用审查功能，例如 Sourcery 生成的拉取请求摘要、评审者指南等。
- 更改审查语言。
- 添加、删除或编辑自定义审查说明。
- 调整其他审查设置。
#### 获得帮助
- [联系我们的支持团队](mailto:support@sourcery.ai) 提出问题或反馈。
- 访问我们的 [文档](https://docs.sourcery.ai) 获取详细指南和信息。
- 通过在 [X/Twitter](https://x.com/SourceryAI)、[LinkedIn](https://www.linkedin.com/company/sourcery-ai/) 或 [GitHub](https://github.com/sourcery-ai) 上关注我们，与 Sourcery 团队保持联系。
</details>Original review guide in EnglishReviewer's Guide by SourceryThis pull request restricts access to several commands by adding an admin permission requirement. This prevents users with lower permissions from disabling default plugins and causing other issues. No diagrams generated as the changes look simple and do not need a visual representation. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
There was a problem hiding this comment.
嘿 @Futureppo - 我已经查看了你的更改 - 这里有一些反馈：
总体评论：
- 考虑将权限分配组合在一起，以提高可读性。
- 看起来你正在向许多命令添加管理员权限 - 仔细检查这是否真的是你想要的。
以下是我在审查期间查看的内容
- 🟢 一般问题：一切看起来都很好
- 🟢 安全性：一切看起来都很好
- 🟢 测试：一切看起来都很好
- 🟢 复杂性：一切看起来都很好
- 🟢 文档：一切看起来都很好
帮助我更有用！请点击每个评论上的 👍 或 👎，我将使用反馈来改进你的评论。
Original comment in English
Hey @Futureppo - I've reviewed your changes - here's some feedback:
Overall Comments:
- Consider grouping the permission assignments together for better readability.
- It looks like you're adding admin permissions to many commands - double check that this is really what you want.
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation
部分指令权限较低，用户可关闭默认插件astrbot以及其他功能导致异常（如gewe_logout指令）。
Modifications
限制部分指令管理员权限
Check
好的，这是翻译成中文的 pull request 总结：
Sourcery 总结
限制敏感命令为仅管理员权限，以防止未经授权的访问和潜在的系统滥用
Bug 修复:
增强功能:
Original summary in English
Summary by Sourcery
Restrict sensitive commands to admin-only permissions to prevent unauthorized access and potential system misuse
Bug Fixes:
Enhancements: