Properly handle multiple trusted certs

Fixes #105

Author: pontusmelke

src/v1/internal/ch-node.js

@@ -116,7 +116,7 @@ const TrustStrategy = {
     }
 
     let tlsOpts = {
-      ca: opts.trustedCertificates.map(fs.readFileSync),
+      ca: opts.trustedCertificates.map((f) => fs.readFileSync(f)),
       // Because we manually check for this in the connect callback, to give
       // a more helpful error to the user
       rejectUnauthorized: false

test/internal/tls.test.js

@@ -22,7 +22,7 @@ var fs = require("fs");
 var path = require('path');
 var hasFeature = require("../../lib/v1/internal/features");
 
-describe('trust-signed-certificates', function() {
+fdescribe('trust-signed-certificates', function() {
 
   var driver;
 
@@ -65,6 +65,24 @@ describe('trust-signed-certificates', function() {
     driver.session().run( "RETURN 1").then( done );
   });
 
+  it('should handle multiple certificates', function(done) {
+    // Assuming we only run this test on NodeJS with TOFU support
+    if( !NodeChannel.available ) {
+      done();
+      return;
+    }
+
+    // Given
+    driver = neo4j.driver("bolt://localhost", neo4j.auth.basic("neo4j", "neo4j"), {
+      encrypted: true,
+      trust: "TRUST_SIGNED_CERTIFICATES",
+      trustedCertificates: ["build/neo4j/certificates/neo4j.cert", "build/neo4j/certificates/neo4j.cert"]
+    });
+
+    // When
+    driver.session().run( "RETURN 1").then( done );
+  });
+
   afterEach(function(){
     if( driver ) {
       driver.close();