echoecho - npm Package Compare versions

Comparing version 0.1.8 to 0.1.9

lib/echo.js

@@ -80,3 +80,3 @@ /*
     },
-    serve: function (req, res) {
+    serve: function (req, res, config) {
         var self = this,
@@ -87,7 +87,14 @@ base = this.validate(req),
             query,
-            data;
+            data,
+            dirroot = __dirname;
 
+        config = config || {};
+
+        if (config.dirroot) {
+            dirroot = config.dirroot;
+        }
+
         if (base && base !== true) {
             if (this.scheme[base]) {
-                query  = parse(req.url).query,
+                query  = parse(req.url).query;
                 parsed = qs.parse(query);
@@ -98,4 +105,5 @@
                 } else if (parsed.file) {
-                    filepath = path.join(__dirname, parsed.file);
+                    filepath = path.join(dirroot, parsed.file);
                     fs.readFile(filepath, function (err, data) {
+                        /*istanbul ignore next*/
                         if (err) {
@@ -102,0 +110,0 @@ self.scheme.status(404, res);

package.json

@@ -5,3 +5,3 @@ {
   "author": "Dav Glass <[e-mail]>",
-  "version": "0.1.8",
+  "version": "0.1.9",
   "devDependencies": {
@@ -8,0 +8,0 @@ "vows": "*",

tests/tests.js

@@ -14,12 +14,16 @@ var vows = require('vows'),
 var server = http.createServer(function(req, res) {
+    var conf = {
+        dirroot: __dirname
+    };
     if (req.url.indexOf('-express') > -1) {
+        conf = null;
         var p = parse(req.url);
         req.url = req.url.replace('-express', '');
         req.body = qs.parse(p.query);
-        echoecho.serve(req, res);
+        echoecho.serve(req, res, conf);
     } else if (echoecho.handle(req)) {
-        echoecho.serve(req, res);
+        echoecho.serve(req, res, conf);
     } else {
         if (req.url.indexOf('skipthisrequest') > -1) {
-            echoecho.serve(req, res);
+            echoecho.serve(req, res, conf);
         } else {
@@ -838,3 +842,3 @@ res.writeHead(200);
 function assertFileParam(route, method) {
-    var path = '/foo/bar/baz/echo/' + route + '?file=../tests/fixtures/file.json'
+    var path = '/foo/bar/baz/echo/' + route + '?file=fixtures/file.json',
         context = {
@@ -851,3 +855,3 @@ topic: function() {
         assert.equal(res.body, file);
-    }
+    };
 
@@ -854,0 +858,0 @@ return context;

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

49405

increased by0.58%

Lines of code

1194

increased by0.84%

No dependency changes