fix: set URL attribute after upload succeed (#640)

Author: sdjdd

src/uploader/cos.js

@@ -2,9 +2,6 @@ const { getAdapter } = require('../adapter');
 const debug = require('debug')('cos');
 
 module.exports = function(uploadInfo, data, file, saveOptions = {}) {
-  file.attributes.url = uploadInfo.url;
-  file._bucket = uploadInfo.bucket;
-  file.id = uploadInfo.objectId;
   const url =
     uploadInfo.upload_url + '?sign=' + encodeURIComponent(uploadInfo.token);
   const fileFormData = {
@@ -29,6 +26,9 @@ module.exports = function(uploadInfo, data, file, saveOptions = {}) {
         error.response = response;
         throw error;
       }
+      file.attributes.url = uploadInfo.url;
+      file._bucket = uploadInfo.bucket;
+      file.id = uploadInfo.objectId;
       return file;
     },
     error => {

src/uploader/qiniu.js

@@ -2,9 +2,6 @@ const { getAdapter } = require('../adapter');
 const debug = require('debug')('leancloud:qiniu');
 
 module.exports = function(uploadInfo, data, file, saveOptions = {}) {
-  file.attributes.url = uploadInfo.url;
-  file._bucket = uploadInfo.bucket;
-  file.id = uploadInfo.objectId;
   // Get the uptoken to upload files to qiniu.
   const uptoken = uploadInfo.token;
   const url = uploadInfo.upload_url || 'https://upload.qiniup.com';
@@ -40,6 +37,9 @@ module.exports = function(uploadInfo, data, file, saveOptions = {}) {
         error.response = response;
         throw error;
       }
+      file.attributes.url = uploadInfo.url;
+      file._bucket = uploadInfo.bucket;
+      file.id = uploadInfo.objectId;
       return file;
     },
     error => {

src/uploader/s3.js

@@ -2,10 +2,6 @@ const _ = require('underscore');
 const ajax = require('../utils/ajax');
 
 module.exports = function upload(uploadInfo, data, file, saveOptions = {}) {
-  file.attributes.url = uploadInfo.url;
-  file._bucket = uploadInfo.bucket;
-  file.id = uploadInfo.objectId;
-
   /* NODE-ONLY:start */
   if (data instanceof require('stream')) {
     // data.pipe(req);
@@ -27,5 +23,10 @@ module.exports = function upload(uploadInfo, data, file, saveOptions = {}) {
       file._uploadHeaders
     ),
     onprogress: saveOptions.onprogress,
-  }).then(() => file);
+  }).then(() => {
+    file.attributes.url = uploadInfo.url;
+    file._bucket = uploadInfo.bucket;
+    file.id = uploadInfo.objectId;
+    return file;
+  });
 };