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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade Qingstor SDK and support private region #1303
Upgrade Qingstor SDK and support private region #1303
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1303 +/- ##
==========================================
+ Coverage 59.32% 59.39% +0.07%
==========================================
Files 126 126
Lines 21466 21470 +4
==========================================
+ Hits 12734 12752 +18
+ Misses 7201 7186 -15
- Partials 1531 1532 +1
Continue to review full report at Codecov.
|
bucketName, zone, host = hostParts[0], "", hostParts[1] | ||
} else { | ||
hostParts := strings.SplitN(uri.Host, ".", 3) | ||
bucketName, zone, host = hostParts[0], hostParts[1], hostParts[2] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should the zone be part of host? Please add an example here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is old logic and has not changed.
endpoint="http://zhjijian-test.sh1a.qingstor.com"
The zone is indeed part of the host.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I meant the host
should be sh1a.qingstor.com
, not qingstor.com
, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No description provided.