Skip to content

Commit

Permalink
Merge pull request #146 from livehigh/feat/getObject-support-useAccel…
Browse files Browse the repository at this point in the history
…erate

Feat/get object support use accelerate
  • Loading branch information
carsonxu authored Apr 14, 2022
2 parents e513768 + 59e79fd commit 84d4f78
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 7 deletions.
8 changes: 5 additions & 3 deletions dist/cos-js-sdk-v5.js
Original file line number Diff line number Diff line change
Expand Up @@ -7871,12 +7871,13 @@ function getAuth(params) {
*/
function getObjectUrl(params, callback) {
var self = this;
var useAccelerate = params.UseAccelerate === undefined ? self.options.UseAccelerate : params.UseAccelerate;
var url = getUrl({
ForcePathStyle: self.options.ForcePathStyle,
protocol: params.Protocol || self.options.Protocol,
domain: params.Domain || self.options.Domain,
bucket: params.Bucket,
region: params.Region,
region: useAccelerate ? 'accelerate' : params.Region,
object: params.Key
});

Expand All @@ -7896,7 +7897,7 @@ function getObjectUrl(params, callback) {
}

// 签名加上 Host,避免跨桶访问
var SignHost = getSignHost.call(this, { Bucket: params.Bucket, Region: params.Region, Url: url });
var SignHost = getSignHost.call(this, { Bucket: params.Bucket, Region: params.Region, UseAccelerate: params.UseAccelerate, Url: url });
var AuthData = getAuthorizationAsync.call(this, {
Action: (params.Method || '').toUpperCase() === 'PUT' ? 'name/cos:PutObject' : 'name/cos:GetObject',
Bucket: params.Bucket || '',
Expand Down Expand Up @@ -8057,12 +8058,13 @@ function getUrl(params) {

var getSignHost = function (opt) {
if (!opt.Bucket || !opt.Region) return '';
var useAccelerate = opt.UseAccelerate === undefined ? this.options.UseAccelerate : opt.UseAccelerate;
var url = opt.Url || getUrl({
ForcePathStyle: this.options.ForcePathStyle,
protocol: this.options.Protocol,
domain: this.options.Domain,
bucket: opt.Bucket,
region: this.options.UseAccelerate ? 'accelerate' : opt.Region
region: useAccelerate ? 'accelerate' : opt.Region
});
var urlHost = url.replace(/^https?:\/\/([^/]+)(\/.*)?$/, '$1');
var standardHostReg = new RegExp('^([a-z\\d-]+-\\d+\\.)?(cos|cosv6|ci|pic)\\.([a-z\\d-]+)\\.myqcloud\\.com$');
Expand Down
2 changes: 1 addition & 1 deletion dist/cos-js-sdk-v5.min.js

Large diffs are not rendered by default.

8 changes: 5 additions & 3 deletions src/base.js
Original file line number Diff line number Diff line change
Expand Up @@ -2991,12 +2991,13 @@ function getAuth(params) {
*/
function getObjectUrl(params, callback) {
var self = this;
var useAccelerate = params.UseAccelerate === undefined ? self.options.UseAccelerate : params.UseAccelerate;
var url = getUrl({
ForcePathStyle: self.options.ForcePathStyle,
protocol: params.Protocol || self.options.Protocol,
domain: params.Domain || self.options.Domain,
bucket: params.Bucket,
region: params.Region,
region: useAccelerate ? 'accelerate' : params.Region,
object: params.Key,
});

Expand All @@ -3016,7 +3017,7 @@ function getObjectUrl(params, callback) {
}

// 签名加上 Host,避免跨桶访问
var SignHost = getSignHost.call(this, {Bucket: params.Bucket, Region: params.Region, Url: url});
var SignHost = getSignHost.call(this, {Bucket: params.Bucket, Region: params.Region, UseAccelerate: params.UseAccelerate, Url: url});
var AuthData = getAuthorizationAsync.call(this, {
Action: ((params.Method || '').toUpperCase() === 'PUT' ? 'name/cos:PutObject' : 'name/cos:GetObject'),
Bucket: params.Bucket || '',
Expand Down Expand Up @@ -3186,12 +3187,13 @@ function getUrl(params) {

var getSignHost = function (opt) {
if (!opt.Bucket || !opt.Region) return '';
var useAccelerate = opt.UseAccelerate === undefined ? this.options.UseAccelerate : opt.UseAccelerate;
var url = opt.Url || getUrl({
ForcePathStyle: this.options.ForcePathStyle,
protocol: this.options.Protocol,
domain: this.options.Domain,
bucket: opt.Bucket,
region: this.options.UseAccelerate ? 'accelerate' : opt.Region,
region: useAccelerate ? 'accelerate' : opt.Region,
});
var urlHost = url.replace(/^https?:\/\/([^/]+)(\/.*)?$/, '$1');
var standardHostReg = new RegExp('^([a-z\\d-]+-\\d+\\.)?(cos|cosv6|ci|pic)\\.([a-z\\d-]+)\\.myqcloud\\.com$');
Expand Down

0 comments on commit 84d4f78

Please sign in to comment.