Merge pull request #136 from qiniu/develop

release 6.1.6

Author: longbai

CHANGELOG.md

@@ -1,5 +1,12 @@
 ## CHANGE LOG
 
+### v6.1.6
+
+2014-10-31
+
+- 增加fsizelimit, insertonly
+
+
 ### v6.1.5
 
 2014-7-23 issue [#111](https://github.com/qiniu/nodejs-sdk/pull/111)

README.md

@@ -41,7 +41,7 @@ $ npm install qiniu
 
 ## 许可证
 
-Copyright (c) 2013 qiniu.com
+Copyright (c) 2014 qiniu.com
 
 基于 MIT 协议发布:
 

docs/README.md

@@ -540,7 +540,7 @@ func pfop(bucketName, keyName, fops, opts, callback);
 
 // 例子
 // pfop
-      qiniu.fop.pfop(TEST_BUCKET, keys[0], 'avinfo', {notifyUrl: 'www.test.com', force: true}, function(err, ret) {
+      qiniu.fop.pfop(TEST_BUCKET, keys[0], 'avinfo', {notifyURL: 'www.test.com', force: true}, function(err, ret) {
         ret.should.have.keys('persistentId');
         done();
       });

package.json

@@ -1,6 +1,6 @@
 {
   "name": "qiniu",
-  "version": "6.1.5",
+  "version": "6.1.6",
   "description": "Node wrapper for Qiniu Resource (Cloud) Storage API",
   "main": "index.js",
   "directories": {

qiniu/conf.js

@@ -6,7 +6,7 @@ exports.ACCESS_KEY = '<PLEASE APPLY YOUR ACCESS KEY>';
 exports.SECRET_KEY = '<DONT SEND YOUR SECRET KEY TO ANYONE>';
 
 var pkg = JSON.parse(fs.readFileSync(path.join(__dirname, '../', 'package.json')));
-var ua = function(){
+var ua = function() {
     return 'QiniuNodejs/' + pkg.version + ' (' + os.type() + '; ' + os.platform() + '; ' + os.arch() + '; )';
 }
 
@@ -15,4 +15,5 @@ exports.USER_AGENT = ua();
 exports.UP_HOST = 'http://upload.qiniu.com';
 exports.RS_HOST = 'http://rs.qbox.me';
 exports.RSF_HOST = 'http://rsf.qbox.me';
+exports.API_HOST = 'http://api.qiniu.com';
 exports.RPC_TIMEOUT = 3600000; // default rpc timeout: one hour

qiniu/fop.js

@@ -1,5 +1,6 @@
 var util = require('./util');
 var rpc = require('./rpc');
+var conf = require('./conf');
 
 var querystring = require('querystring');
 
@@ -17,7 +18,7 @@ function ImageView(mode, width, height, quality, format) {
 }
 
 ImageView.prototype.makeRequest = function(url) {
-  url += '?imageView/' + this.mode;
+  url += '?imageView2/' + this.mode;
 
   if (this.width > 0) {
     url += '/w/' + this.width;
@@ -38,15 +39,13 @@ ImageView.prototype.makeRequest = function(url) {
   return url;
 }
 
-function ImageInfo() {
-}
+function ImageInfo() {}
 
 ImageInfo.prototype.makeRequest = function(url) {
   return url + '?imageInfo'
 }
 
-function Exif() {
-}
+function Exif() {}
 
 Exif.prototype.makeRequest = function(url) {
   return url + '?exif'
@@ -70,10 +69,12 @@ function pfop(bucket, key, fops, opts, onret) {
   if (opts.force) {
     param.force = 1;
   }
+  if (opts.pipeline) {
+    param.pipeline = opts.pipeline;
+  }
 
-  var uri = 'http://api.qiniu.com/pfop/';
+  var uri = conf.API_HOST + '/pfop/';
   var body = querystring.stringify(param);
   var auth = util.generateAccessToken(uri, body);
   rpc.postWithForm(uri, body, auth, onret);
 }
-

qiniu/rs.js

@@ -1,4 +1,3 @@
-
 var url = require('url');
 var crypto = require('crypto');
 var formstream = require('formstream');
@@ -58,6 +57,13 @@ Client.prototype.copy = function(bucketSrc, keySrc, bucketDest, keyDest, onret)
   rpc.postWithoutForm(uri, digest, onret);
 }
 
+Client.prototype.fetch = function(url, bucket, key, onret) {
+  var bucketUri = getEncodedEntryUri(bucket, key);
+  var fetchUrl = util.urlsafeBase64Encode(url);
+  var uri = 'http://iovip.qbox.me/fetch/' + fetchUrl + '/to/' + bucketUri;
+  var digest = util.generateAccessToken(uri, null);
+  rpc.postWithoutForm(uri, digest, onret);
+}
 
 function Entry(hash, fsize, putTime, mimeType, endUser) {
   this.hash = hash || null;
@@ -135,17 +141,35 @@ function getEncodedEntryUri(bucket, key) {
 
 // ----- token --------
 // @gist PutPolicy
-function PutPolicy(scope, callbackUrl, callbackBody, returnUrl, returnBody,
-                  asyncOps, endUser, expires, persistentOps, persistentNotifyUrl) {
-  this.scope = scope || null;
-  this.callbackUrl = callbackUrl || null;
-  this.callbackBody = callbackBody || null;
-  this.returnUrl = returnUrl || null;
-  this.returnBody = returnBody || null;
-  this.endUser = endUser || null;
-  this.expires = expires || 3600;
-  this.persistentOps = persistentOps || null;
-  this.persistentNotifyUrl = persistentNotifyUrl || null;
+function PutPolicy(putPolicyObj) {
+
+  if (typeof putPolicyObj !== 'object') {
+    return false;
+  }
+
+  this.scope = putPolicyObj.scope || null;
+  this.expires = putPolicyObj.expires || 3600;
+  this.insertOnly = putPolicyObj.insertOnly || null;
+
+  this.saveKey = putPolicyObj.saveKey || null;
+  this.endUser = putPolicyObj.endUser || null;
+
+  this.returnUrl = putPolicyObj.returnUrl || null;
+  this.returnBody = putPolicyObj.returnBody || null;
+
+  this.callbackUrl = putPolicyObj.callbackUrl || null;
+  this.callbackHost = putPolicyObj.callbackHost || null;
+  this.callbackBody = putPolicyObj.callbackBody || null;
+
+  this.persistentOps = putPolicyObj.persistentOps || null;
+  this.persistentNotifyUrl = putPolicyObj.persistentNotifyUrl || null;
+  this.persistentPipeline = putPolicyObj.persistentPipeline || null;
+
+  this.fsizeLimit = putPolicyObj.fsizeLimit || null;
+
+  this.detectMime = putPolicyObj.detectMime || null;
+
+  this.mimeLimit = putPolicyObj.mimeLimit || null;
 }
 // @endgist
 
@@ -161,54 +185,18 @@ PutPolicy.prototype.token = function(mac) {
   return uploadToken;
 }
 
-PutPolicy.prototype.getFlags = function(putPolicy) {
+PutPolicy.prototype.getFlags = function() {
   var flags = {};
-  if (this.scope != null) {
-    flags['scope'] = this.scope;
-  }
-  if (this.callbackUrl != null) {
-    flags['callbackUrl'] = this.callbackUrl;
-  }
-  if (this.callbackBody != null) {
-    flags['callbackBody'] = this.callbackBody;
-  }
-  if (this.returnUrl != null) {
-    flags['returnUrl'] = this.returnUrl;
-  }
-  if (this.returnBody != null) {
-    flags['returnBody'] = this.returnBody;
-  }
-  if (this.endUser != null) {
-    flags['endUser'] = this.endUser;
-  }
-  if (this.persistentOps != null) {
-    flags['persistentOps'] = this.persistentOps;
-  }
-  if (this.persistentNotifyUrl != null) {
-    flags['persistentNotifyUrl'] = this.persistentNotifyUrl;
-  }
-  if (this.persistentPipeline != null) {
-    flags['persistentPipeline'] = this.persistentPipeline;
-  }
-  if (this.mimeLimit != null) {
-    flags['mimeLimit'] = this.mimeLimit;
-  }
-  if (this.insertOnly != null) {
-    flags['insertOnly'] = this.insertOnly;
-  }
-  if (this.detectMime != null) {
-    flags['detectMime'] = this.detectMime;
-  }
-  if (this.saveKey != null) {
-    flags['saveKey'] = this.saveKey;
+  var attrs = ['scope', 'insertOnly', 'saveKey', 'endUser', 'returnUrl', 'returnBody', 'callbackUrl', 'callbackHost', 'callbackBody', 'callbackBodyType', 'callbackFetchKey', 'persistentOps', 'persistentNotifyUrl', 'persistentPipeline', 'fsizeLimit', 'detectMime', 'mimeLimit'];
+
+  for (var i = attrs.length - 1; i >= 0; i--) {
+    if (this[attrs[i]] !== null) {
+      flags[attrs[i]] = this[attrs[i]];
+    }
   }
+
   flags['deadline'] = this.expires + Math.floor(Date.now() / 1000);
-  if (this.fsizeLimit != null) {
-    flags['fsizeLimit'] = this.fsizeLimit;
-  }
-  if (this.insertOnly != null) {
-    flags['insertOnly'] = this.insertOnly;
-  }
+
   return flags;
 }
 
@@ -239,5 +227,5 @@ GetPolicy.prototype.makeRequest = function(baseUrl, mac) {
 // query like '-thumbnail', '?imageMogr2/thumbnail/960x' and so on
 function makeBaseUrl(domain, key, query) {
   key = new Buffer(key);
-  return 'http://' + domain + '/' + querystring.escape(key) + (query||'');
+  return 'http://' + domain + '/' + querystring.escape(key) + (query || '');
 }

test/fop.test.js

@@ -19,12 +19,12 @@ describe('test start step 0', function() {
         iv.height = 100;
         iv.width = 40;
         var returl = iv.makeRequest(pic);
-        returl.should.equal(pic + '?imageView/1/w/40/h/100');
+        returl.should.equal(pic + '?imageView2/1/w/40/h/100');
 
         iv.quality = 20;
         iv.format = 'jpg';
         returl = iv.makeRequest(pic);
-        returl.should.equal(pic + '?imageView/1/w/40/h/100/q/20/format/jpg');
+        returl.should.equal(pic + '?imageView2/1/w/40/h/100/q/20/format/jpg');
         done();
       });
     });

test/io.test.js

@@ -44,8 +44,10 @@ describe('test start step1:', function() {
     describe('upload#', function() {
       var uptoken = null;
       beforeEach(function(done) {
-        var putPolicy = new qiniu.rs.PutPolicy(TEST_BUCKET);
-        uptoken = putPolicy.token();
+        var putPolicy = new qiniu.rs.PutPolicy({
+          scope: TEST_BUCKET
+        });
+	uptoken = putPolicy.token();
         done();
       });
 

test/rs.test.js

@@ -25,7 +25,9 @@ describe('test start step2:', function() {
     describe('single file handle', function() {
 
       before(function(done) {
-        var putPolicy = new qiniu.rs.PutPolicy(TEST_BUCKET);
+        var putPolicy = new qiniu.rs.PutPolicy({
+          scope: TEST_BUCKET
+        });
         var uptoken = putPolicy.token();
         qiniu.io.putFile(uptoken, logo2, imageFile, null, function(err, ret) {
           should.not.exist(err);