Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

文件上传增加S3协议的OSS支持#328

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
liujiang157 wants to merge9 commits intohs-web:master
base:master
Choose a base branch
Loading
fromliujiang157:feature/oss-support

Conversation

@liujiang157
Copy link

切换本地和oss通过环境变了file.storage=local/s3控制
s3协议的相关配置均在S3StorageProperties类中

@CLAassistant
Copy link

CLAassistant commentedJun 16, 2025
edited
Loading

CLA assistant check
All committers have signed the CLA.

liujiang157 and mengnankkkk reacted with thumbs up emoji

@liujiang157
Copy link
Author

重新调整了代码,走本地还是 S3 通过 "hsweb.file.storage"来控制,S3相关配置也遵循之前配置属性的命名基础上添加。控制
如果是S3则:
System.setProperty("hsweb.file.storage", "s3");
如果本地则设置如下或者不设置:
System.setProperty("hsweb.file.storage", "local");

S3的相关配置:
hsweb.file.upload.s3.endpoint
hsweb.file.upload.s3.region
hsweb.file.upload.s3.accessKey
hsweb.file.upload.s3.secretKey
hsweb.file.upload.s3.bucket

单元测试成功之后我删除了accessKey和secretKey,其余问题也改掉了

修改error应当由Mono传递,而不是抛出异常移除非必要Condition条件
采用buffer.asInputStream访问文件移除冗余代码
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@zhou-haozhou-haozhou-hao left review comments

+1 more reviewer

@mengnankkkkmengnankkkkmengnankkkk approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@liujiang157@CLAassistant@zhou-hao@mengnankkkk

[8]ページ先頭

©2009-2025 Movatter.jp