Fix edge case where some files occassionaly would not load in

motdotla · motdotla · commit ba9232831da4 · 2015-02-12T21:54:28.000-05:00
diff --git a/lib/main.js b/lib/main.js
@@ -113,13 +113,12 @@ Lambda.prototype._zip = function(program, codeDirectory, callback) {
   console.log('Zipping repo. This might take up to 30 seconds');
   var files = wrench.readdirSyncRecursive(codeDirectory);
   files.forEach(function(file) {
-    try {
-      var isFile = fs.lstatSync(file).isFile();
-      if (isFile) {
-        var content = fs.readFileSync(file);
-        zip.file(file, content);
-      }
-    } catch(err) {}
+    var filePath = [codeDirectory, file].join('/');
+    var isFile = fs.lstatSync(filePath).isFile();
+    if (isFile) {
+      var content = fs.readFileSync(filePath);
+      zip.file(file, content);
+    }
   });
   
   var data = zip.generate(options);
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "node-lambda",
-  "version": "0.3.11",
+  "version": "0.3.12",
   "description": "Command line tool for locally running and remotely deploying your node.js applications to Amazon Lambda.",
   "main": "lib/main.js",
   "directories": {
diff --git a/test/main.js b/test/main.js
@@ -36,7 +36,7 @@ describe('node-lambda', function() {
   });
 
   it('version should be set', function() {
-    assert.equal(lambda.version, '0.3.11');
+    assert.equal(lambda.version, '0.3.12');
   });
 
   describe('_params', function() {
@@ -86,7 +86,7 @@ describe('node-lambda', function() {
       });
     });
 
-    it('_npm adds node_mdules', function(done) {
+    it('_npm adds node_modules', function(done) {
       this.timeout(5000); // give it time to build the node modules
 
       lambda._npmInstall(program, codeDirectory, function(err, result) {

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "node-lambda",`
`3`		`- "version": "0.3.11",`
	`3`	`+ "version": "0.3.12",`
`4`	`4`	`"description": "Command line tool for locally running and remotely deploying your node.js applications to Amazon Lambda.",`
`5`	`5`	`"main": "lib/main.js",`
`6`	`6`	`"directories": {`