Merge pull request #11 from DullReferenceException/abs-cert-path-fix

Fix cert path resolution with absolute paths

Author: indexzero

.gitignore

@@ -13,3 +13,5 @@ results
 
 npm-debug.log
 node_modules
+
+.idea

index.js

@@ -146,11 +146,11 @@ module.exports = function createServers(options, listening) {
       //
       // Load default SSL key, cert and ca(s).
       //
-      key:  fs.readFileSync(path.join(ssl.root, ssl.key)),
-      cert: fs.readFileSync(path.join(ssl.root, ssl.cert)),
+      key:  fs.readFileSync(path.resolve(ssl.root, ssl.key)),
+      cert: fs.readFileSync(path.resolve(ssl.root, ssl.cert)),
       ca:   ssl.ca && ssl.ca.map(
         function (file) {
-          return fs.readFileSync(path.join(ssl.root, file));
+          return fs.readFileSync(path.resolve(ssl.root, file));
         }
       ),
       //

test/create-servers-test.js

@@ -44,7 +44,25 @@ test('only https', function (t) {
     },
     handler: fend
   }, function (err, servers) {
-    console.dir(err);
+    t.error(err);
+    t.equals(typeof servers, 'object');
+    t.equals(typeof servers.https, 'object');
+    servers.https.close();
+  });
+});
+
+test('absolute cert path resolution', function (t) {
+  t.plan(3);
+  createServers({
+    log: console.log,
+    https: {
+      port: 3456,
+      root: '/',
+      cert: path.resolve(__dirname, 'fixtures', 'agent2-cert.pem'),
+      key:  path.resolve(__dirname, 'fixtures', 'agent2-key.pem')
+    },
+    handler: fend
+  }, function (err, servers) {
     t.error(err);
     t.equals(typeof servers, 'object');
     t.equals(typeof servers.https, 'object');
@@ -65,7 +83,6 @@ test('http && https', function (t) {
     },
     handler: fend
   }, function (err, servers) {
-    console.dir(err);
     t.error(err);
     t.equals(typeof servers, 'object');
     t.equals(typeof servers.http, 'object');
@@ -115,7 +132,6 @@ test('http && https with different handlers', function (t) {
       key:  'agent2-key.pem'
     },
   }, function (err, servers) {
-    console.dir(err);
     t.error(err);
     t.equals(typeof servers, 'object');
     t.equals(typeof servers.http, 'object');
@@ -131,7 +147,6 @@ test('http && https with different handlers', function (t) {
       http: '9876',
       handler: fend
     }, function (err, servers) {
-      console.dir(err);
       t.error(err);
       t.equals(typeof servers, 'object');
       t.equals(typeof servers.http, 'object');
@@ -149,7 +164,6 @@ test('http && https with different handlers', function (t) {
       },
       handler: fend
     }, function (err, servers) {
-      console.dir(err);
       t.error(err);
       t.equals(typeof servers, 'object');
       t.equals(typeof servers.http, 'object');