webdav - npm Package Compare versions

Comparing version 0.1.0 to 0.1.1

package.json

 {
   "name": "webdav",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "WebDAV client for NodeJS",
@@ -5,0 +5,0 @@ "main": "source/index.js",

README.md

@@ -6,2 +6,4 @@ # WebDAV client
 
+[![NPM](https://nodei.co/npm/webdav.png?downloads=true&downloadRank=true&stars=true)](https://www.npmjs.com/package/webdav)
+
 ## About
@@ -8,0 +10,0 @@ This client was branched from [webdav-fs](https://github.com/perry-mitchell/webdav-fs) as the core functionality deserved its own repository. As **webdav-fs**' API was designed to resemble NodeJS' fs API, little could be done to improve the adapter interface for regular use.

source/adapter/alter.js

 var fetch = require("node-fetch");
 
+var responseHandlers = require("./response.js");
+
 module.exports = {
@@ -7,4 +9,5 @@
         return fetch(url + targetPath, {
-            method: "DELETE"
-        });
+                method: "DELETE"
+            })
+            .then(responseHandlers.handleResponseCode);
     },
@@ -14,9 +17,10 @@
         return fetch(url + filePath, {
-            method: "MOVE",
-            headers: {
-                Destination: url + targetFilePath
-            }
-        });
+                method: "MOVE",
+                headers: {
+                    Destination: url + targetFilePath
+                }
+            })
+            .then(responseHandlers.handleResponseCode);
     }
 
 };

source/adapter/get.js

 var fetch = require("node-fetch"),
     xml2js = require("xml2js");
 
-var parsing = require("./parse.js");
+var parsing = require("./parse.js"),
+    responseHandlers = require("./response.js");
 
@@ -20,2 +21,3 @@ module.exports = {
             )
+            .then(responseHandlers.handleResponseCode)
             .then(function(res) {
@@ -42,2 +44,3 @@ return res.text();
         return fetch(url + filePath)
+            .then(responseHandlers.handleResponseCode)
             .then(function(res) {
@@ -55,2 +58,3 @@ return res.buffer();
             })
+            .then(responseHandlers.handleResponseCode)
             .then(function(res) {
@@ -81,2 +85,3 @@ return res.text();
         return fetch(url + filePath)
+            .then(responseHandlers.handleResponseCode)
             .then(function(res) {
@@ -83,0 +88,0 @@ return res.text();

source/adapter/put.js

 var fetch = require("node-fetch");
 
+var responseHandlers = require("./response.js");
+
 module.exports = {
@@ -7,4 +9,5 @@
         return fetch(url + directoryPath, {
-            method: "MKCOL"
-        });
+                method: "MKCOL"
+            })
+            .then(responseHandlers.handleResponseCode);
     },
@@ -14,9 +17,10 @@
         return fetch(url + filePath, {
-            method: "PUT",
-            headers: {
-                "Content-Type": "application/octet-stream",
-                "Content-Length": data.length
-            },
-            body: data
-        });
+                method: "PUT",
+                headers: {
+                    "Content-Type": "application/octet-stream",
+                    "Content-Length": data.length
+                },
+                body: data
+            })
+            .then(responseHandlers.handleResponseCode);
     },
@@ -26,11 +30,12 @@
         return fetch(url + filePath, {
-            method: "PUT",
-            headers: {
-                "Content-Type": "text/plain",
-                "Content-Length": text.length
-            },
-            body: text
-        });
+                method: "PUT",
+                headers: {
+                    "Content-Type": "text/plain",
+                    "Content-Length": text.length
+                },
+                body: text
+            })
+            .then(responseHandlers.handleResponseCode);
     }
 
 };

source/url.js

@@ -0,1 +1,3 @@
+var queryString = require("querystring");
+
 module.exports = {
@@ -5,4 +7,6 @@
         return (username && username.length > 0) ?
-            url.replace(/(https?:\/\/)/i, "$1" + username + ":" + password + "@") :
-            url;
+            url.replace(
+                /(https?:\/\/)/i,
+                "$1" + queryString.escape(username) + ":" + queryString.escape(password) + "@"
+            ) : url;
     },
@@ -9,0 +13,0 @@

No alert changes

Improved metrics

Total package byte prevSize

31537

increased by4.7%

Number of package files

21

increased by10.53%

Lines of code

608

increased by4.11%

Number of lines in readme file

174

increased by1.16%

No dependency changes