release v2.2.2

Author: timzaak

client/src/upload_files.rs

@@ -195,11 +195,8 @@ async fn get_upload_version(api: &API, domain: &str, version: Option<u32>) -> an
         Ok(version)
     } else {
         let resp = api.get_upload_position(domain).await?;
-        match resp.status {
-            GetDomainPositionStatus::NewDomain => {
-                println!("domain:{} is new in server!", domain);
-            }
-            _ => {}
+        if resp.status == GetDomainPositionStatus::NewDomain {
+            println!("domain:{} is new in server!", domain);
         };
         Ok(resp.version)
     }

docs/develop/change-log.md

@@ -7,6 +7,7 @@
 - [x] fix: fix cors check for normal http request.
 - [x] fix: update version of one SPA one domain would cause deadlock.
 - [x] feat: support index.html alias "","/" for multiple SPA one domain.
+- [x] improve: JS SDK keep same logic with command line, and improve error information. release JS SDK 2.2.2
 - [x] improve: more integration tests.
 
 ### Version 2.2.1

jsclient/package.json

@@ -1,6 +1,6 @@
 {
   "name": "spa-client",
-  "version": "2.0.0",
+  "version": "2.2.2",
   "description": "js sdk for spa-server",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",

jsclient/src/index.ts

@@ -55,8 +55,8 @@ export default class SPAClient {
                 "Authorization": `Bearer ${config.authToken}`
             }
         })
-        this.http.interceptors.response.use((resp) => resp,(error) => {
-            return Promise.reject( error.cause)
+        this.http.interceptors.response.use((resp) => resp, (error) => {
+            return Promise.reject(error.cause)
         })
     }
 
@@ -117,6 +117,10 @@ export default class SPAClient {
         if (!(fs.existsSync(absolutePath) && fs.statSync(absolutePath).isDirectory())) {
             throw `path:${path} is not directory or does not exists`
         }
+        const files = await walkdir.async(absolutePath, {return_object: true})
+        if (Object.keys(files).length == 0) {
+            throw `path:${path} has no files`
+        }
         let realVersion: number;
         if (!version) {
             const positionResp = await this.getUploadPosition(domain)
@@ -136,7 +140,6 @@ export default class SPAClient {
             result[item.path] = item
             return result
         }, {} as { [key: string]: ShortMetaData })
-        const files = await walkdir.async(absolutePath, {return_object: true})
         const uploadingFiles = Object.keys(files).reduce((result, filePath) => {
             const fileStat = files[filePath]
             if (fileStat.isFile()) {
@@ -151,7 +154,9 @@ export default class SPAClient {
 
         }, [] as { key: string, filePath: string }[])
         if (!uploadingFiles.length) {
-            throw "There is no file to uploading"
+            console.log(chalk.green("all files already upload"))
+            await this.changeUploadingStatus(domain, realVersion, UploadingStatus.Finish)
+            return
         }
         console.log(chalk.green(`there are ${uploadingFiles.length} files to upload`))
         await this.changeUploadingStatus(domain, realVersion, UploadingStatus.Uploading)