Fix: if there are duplicate hosts in the known_hosts file, we return on the first one and not call the callback on loadFingerprint more than once

Author: Stefan-Gabriel Muscalu

src/v1/internal/ch-node.js

@@ -45,7 +45,7 @@ function loadFingerprint( serverId, knownHostsPath, cb ) {
   require('readline').createInterface({
     input: fs.createReadStream(knownHostsPath)
   }).on('line', (line)  => {
-    if( line.startsWith( serverId )) {
+    if( !found && line.startsWith( serverId )) {
       found = true;
       cb( line.split(" ")[1] );
     }

test/internal/tls.test.js

@@ -76,6 +76,41 @@ describe('trust-on-first-use', function() {
 
   var driver;
 
+  it('should not throw an error if the host file contains two host duplicates', function(done) {
+    'use strict';
+    // Assuming we only run this test on NodeJS with TOFU support
+    if( !hasFeature("trust_on_first_use") ) {
+      done();
+      return;
+    }
+
+    // Given
+    var knownHostsPath = "build/known_hosts";
+    if( fs.existsSync(knownHostsPath) ) {
+      fs.unlinkSync(knownHostsPath);
+    }
+
+    fs.writeFileSync(
+      knownHostsPath,
+      'localhost:7687 abcd\n' +
+      'localhost:7687 abcd'
+    );
+
+    driver = neo4j.driver("bolt://localhost", neo4j.auth.basic("neo4j", "neo4j"), {
+      encrypted: true,
+      trust: "TRUST_ON_FIRST_USE",
+      knownHosts: knownHostsPath
+    });
+
+    // When
+    driver.session().run("RETURN true AS a").then( function(data) {
+      // Then we get to here.
+      // And then the known_hosts file should have correct contents
+      expect( data.records[0].get('a') ).toBe( true );
+      done();
+    });
+  });
+
   it('should accept previously un-seen hosts', function(done) {
     // Assuming we only run this test on NodeJS with TOFU support
     if( !hasFeature("trust_on_first_use") ) {