Skip to content

refactor(services/s3): Check header first for region detect#2691

Merged
Xuanwo merged 3 commits intomainfrom
polish-detect-region
Jul 23, 2023
Merged

refactor(services/s3): Check header first for region detect#2691
Xuanwo merged 3 commits intomainfrom
polish-detect-region

Conversation

@Xuanwo
Copy link
Copy Markdown
Member

@Xuanwo Xuanwo commented Jul 23, 2023

No description provided.

Signed-off-by: Xuanwo <github@xuanwo.io>
@Xuanwo Xuanwo requested review from ClSlaid and suyanhanx July 23, 2023 08:30
@github-actions github-actions Bot added the releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" label Jul 23, 2023
Signed-off-by: Xuanwo <github@xuanwo.io>
Copy link
Copy Markdown
Member

@suyanhanx suyanhanx left a comment

Choose a reason for hiding this comment

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

Thanks! Let's wating for ci green.

@Xuanwo Xuanwo marked this pull request as draft July 23, 2023 08:42
Signed-off-by: Xuanwo <github@xuanwo.io>
@Xuanwo Xuanwo marked this pull request as ready for review July 23, 2023 08:43
@Xuanwo Xuanwo requested a review from PsiACE as a code owner July 23, 2023 08:43
@Xuanwo Xuanwo merged commit 3ecab57 into main Jul 23, 2023
@Xuanwo Xuanwo deleted the polish-detect-region branch July 23, 2023 08:49
@oowl oowl mentioned this pull request Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants