Skip to content

Fix default S3 region#217

Open
ruibaby wants to merge 1 commit into
mainfrom
fix/default-region-us-east-1
Open

Fix default S3 region#217
ruibaby wants to merge 1 commit into
mainfrom
fix/default-region-us-east-1

Conversation

@ruibaby

@ruibaby ruibaby commented Jun 21, 2026

Copy link
Copy Markdown
Member

Summary

  • Change the default S3 region from Auto to us-east-1
  • Fall back to us-east-1 when the Region field is left blank
  • Update the Region field help text and README guidance

Rationale

Auto is not a valid SigV4 region for S3-compatible services that validate lowercase region names, such as RustFS. us-east-1 is a safer default and Cloudflare R2 aliases it to auto.

Testing

  • ./gradlew test
  • git diff --check

@ruibaby ruibaby marked this pull request as ready for review June 21, 2026 15:16
@ruibaby ruibaby requested a review from longjuan June 22, 2026 07:56
@longjuan

Copy link
Copy Markdown
Member

/lgtm
这似乎是个break change,原来没填但能用的用户更新之后可能变成不能用了?

@longjuan longjuan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

我的又拍云之前使用的是region默认参数,用了这版插件后仍可正常上传。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants