hubot-freight - npm Package Compare versions

Comparing version 0.0.2 to 0.0.3

package.json

 {
   "name": "hubot-freight",
-  "version": "0.0.2",
+  "version": "0.0.3",
   "author": "David Cramer <[e-mail]>",
@@ -5,0 +5,0 @@ "description": "Freight integration for Hubot",

src/scripts/deploy.js

@@ -26,3 +26,3 @@ // Description:
 
-  console.log("Freight: Making request to " + url);
+  console.log("Freight: Making request to %s", url);
 
@@ -33,7 +33,8 @@ msg.http(url)
     .header("Content-Type", "application/json")
-    .post(JSON.stringify(data), function(err, resp, body) {
+    .post(JSON.stringify(data))(function(err, resp, body) {
+      console.log(resp.statusCode);
       if (err) {
         msg.reply("Freight says: " + err);
-        console.error(err);
-      } else if (200 <= resp.statusCode < 400) {  // Or, not an error code.
+        console.error('HTTP %d: %s', resp.statusCode, err);
+      } else if (200 <= resp.statusCode && resp.statusCode < 400) {
         return;
@@ -43,3 +44,3 @@ } else {
         msg.reply("Freight responded with HTTP " + resp.statusCode + ": " + data.error);
-        console.error("HTTP " + resp.statusCode + ": " + body);
+        console.error("HTTP %d: %s", resp.statusCode, body);
       }
@@ -60,3 +61,3 @@ });
 
-  console.log("Freight: Making request to " + url);
+  console.log("Freight: Making request to %s", url);
 
@@ -67,7 +68,7 @@ msg.http(url)
     .header("Content-Type", "application/json")
-    .put(JSON.stringify(data), function(err, resp, body) {
+    .put(JSON.stringify(data))(function(err, resp, body) {
       if (err) {
         msg.reply("Freight says: " + err);
-        console.error(err);
-      } else if (200 <= resp.statusCode < 400) {  // Or, not an error code.
+        console.error('HTTP %d: %s', resp.statusCode, err);
+      } else if (200 <= resp.statusCode && resp.statusCode < 400) {
         return;
@@ -77,3 +78,3 @@ } else {
         msg.reply("Freight responded with HTTP " + resp.statusCode + ": " + data.error);
-        console.error("HTTP " + resp.statusCode + ": " + body);
+        console.error("HTTP %d: %s", resp.statusCode, body);
       }
@@ -80,0 +81,0 @@ });

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

14709

increased by0.55%

Lines of code

78

increased by1.3%

No dependency changes