Skip to content

Commit adc1727

Browse files
committed
Fixes from code review.
1 parent c6ae929 commit adc1727

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

src/v1/internal/ch-node.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ const TrustStrategy = {
129129
" using `neo4j.v1.driver(.., { trustedCertificates:['path/to/certificate.crt']}). This " +
130130
" is a security measure to protect against man-in-the-middle attacks. If you are just trying " +
131131
" Neo4j out and are not concerned about encryption, simply disable it using `encrypted=false` in the driver" +
132-
" options."));
132+
" options. Socket responded with: " + socket.authorizationError));
133133
} else {
134134
onSuccess();
135135
}

test/internal/tls.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ var fs = require("fs");
2222
var path = require('path');
2323
var hasFeature = require("../../lib/v1/internal/features");
2424

25-
fdescribe('trust-signed-certificates', function() {
25+
describe('trust-signed-certificates', function() {
2626

2727
var driver;
2828

@@ -76,7 +76,7 @@ fdescribe('trust-signed-certificates', function() {
7676
driver = neo4j.driver("bolt://localhost", neo4j.auth.basic("neo4j", "neo4j"), {
7777
encrypted: true,
7878
trust: "TRUST_SIGNED_CERTIFICATES",
79-
trustedCertificates: ["build/neo4j/certificates/neo4j.cert", "build/neo4j/certificates/neo4j.cert"]
79+
trustedCertificates: ["build/neo4j/certificates/neo4j.cert", "test/resources/random.certificate"]
8080
});
8181

8282
// When

test/v1/tck/steps/tlssteps.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ module.exports = function () {
7676
"to store this information by setting `knownHosts` to another path in your driver configuration - " +
7777
"and you can disable encryption there as well using `encrypted:false`.";
7878
if (this.error.message !== expected) {
79-
callback(new Error("Given and expected results does not match: " + this.error.message + " Expected " + expected));
79+
callback(new Error("Given and expected results does not match: " + this.error.message + " Expected: " + expected));
8080
} else {
8181
callback();
8282
}
@@ -156,7 +156,7 @@ module.exports = function () {
156156
"certificate, or the server certificate, to the list of certificates trusted by this driver using " +
157157
"`neo4j.v1.driver(.., { trustedCertificates:['path/to/certificate.crt']}). This is a security measure to protect " +
158158
"against man-in-the-middle attacks. If you are just trying Neo4j out and are not concerned about encryption, " +
159-
"simply disable it using `encrypted=false` in the driver options.";
159+
"simply disable it using `encrypted=false` in the driver options. Socket responded with: DEPTH_ZERO_SELF_SIGNED_CERT";
160160
if (this.error.message !== expected) {
161161
callback(new Error("Given and expected results does not match: " + this.error.message + " Expected " + expected));
162162
} else {

0 commit comments

Comments
 (0)