tls: revert accidental API change

socket.authorizationError should always be string. Also make sni test pass.
nodejs · Jul 20, 2012 · 5950db1 · 5950db1
1 parent 4aa09d1
commit 5950db1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/lib/tls.js b/lib/tls.js
@@ -1059,7 +1059,7 @@ function Server(/* [options], listener */) {
       } else {
         var verifyError = pair.ssl.verifyError();
         if (verifyError) {
-          pair.cleartext.authorizationError = verifyError;
+          pair.cleartext.authorizationError = verifyError.message;
 
           if (self.rejectUnauthorized) {
             socket.destroy();

diff --git a/test/simple/test-tls-sni-server-client.js b/test/simple/test-tls-sni-server-client.js
@@ -94,7 +94,9 @@ server.listen(serverPort, startTest);
 function startTest() {
   function connectClient(options, callback) {
     var client = tls.connect(options, function() {
-      clientResults.push(client.authorized);
+      clientResults.push(
+        client.authorizationError &&
+        /Hostname\/IP doesn't/.test(client.authorizationError));
       client.destroy();
 
       callback();