Open
Conversation
zyfjrx
commented
Mar 6, 2026
zyfjrx
commented
- 在导出列表中增加OSSConfig和OSS类
- 新增OSSConfig配置模型，包含endpoint、access_key等字段
- 扩展FileStorageConfig联合类型以支持OSS配置
- 实现OSS存储类，继承Storage基类
- 添加OSS初始化方法，创建认证和Bucket对象
- 实现get、set、delete、get_url等存储操作方法
- 添加私有_get_full_key方法处理键前缀逻辑
- 在create_storage工厂函数中添加OSS实例创建分支
- 使用装饰器class_requires_deps确保oss2依赖可用
- 在导出列表中增加OSSConfig和OSS类 - 新增OSSConfig配置模型，包含endpoint、access_key等字段 - 扩展FileStorageConfig联合类型以支持OSS配置 - 实现OSS存储类，继承Storage基类 - 添加OSS初始化方法，创建认证和Bucket对象 - 实现get、set、delete、get_url等存储操作方法 - 添加私有_get_full_key方法处理键前缀逻辑 - 在create_storage工厂函数中添加OSS实例创建分支 - 使用装饰器class_requires_deps确保oss2依赖可用
|
Thanks for your contribution! |
Bobholamovic
reviewed
Mar 9, 2026
| key = self._get_full_key(key) | ||
| self._bucket.delete_object(key) | ||
|
|
||
| def get_url(self, key: str) -> str: |
|
|
||
| super().__init__() | ||
|
|
||
| # 创建认证对象 |
| return key | ||
|
|
||
|
|
||
| @class_requires_deps("oss2") |
Member
There was a problem hiding this comment.
新增了依赖，可能需要确认一下是否能通过pre-commit检查
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.