Add debug logging for Tight authentication

Makes it easier to diagnose user issues when we can see what the
server and noVNC are trying to negotiate.
This commit is contained in:
Pierre Ossman 2018-06-07 14:57:17 +02:00
parent 13364d70dd
commit e6bad200e4
1 changed files with 6 additions and 0 deletions

View File

@ -1006,6 +1006,8 @@ RFB.prototype = {
serverSupportedTunnelTypes[cap_code] = { vendor: cap_vendor, signature: cap_signature }; serverSupportedTunnelTypes[cap_code] = { vendor: cap_vendor, signature: cap_signature };
} }
Log.Debug("Server Tight tunnel types: " + serverSupportedTunnelTypes);
// choose the notunnel type // choose the notunnel type
if (serverSupportedTunnelTypes[0]) { if (serverSupportedTunnelTypes[0]) {
if (serverSupportedTunnelTypes[0].vendor != clientSupportedTunnelTypes[0].vendor || if (serverSupportedTunnelTypes[0].vendor != clientSupportedTunnelTypes[0].vendor ||
@ -1013,6 +1015,7 @@ RFB.prototype = {
return this._fail("Client's tunnel type had the incorrect " + return this._fail("Client's tunnel type had the incorrect " +
"vendor or signature"); "vendor or signature");
} }
Log.Debug("Selected tunnel type: " + clientSupportedTunnelTypes[0]);
this._sock.send([0, 0, 0, 0]); // use NOTUNNEL this._sock.send([0, 0, 0, 0]); // use NOTUNNEL
return false; // wait until we receive the sub auth count to continue return false; // wait until we receive the sub auth count to continue
} else { } else {
@ -1058,9 +1061,12 @@ RFB.prototype = {
serverSupportedTypes.push(capabilities); serverSupportedTypes.push(capabilities);
} }
Log.Debug("Server Tight authentication types: " + serverSupportedTypes);
for (let authType in clientSupportedTypes) { for (let authType in clientSupportedTypes) {
if (serverSupportedTypes.indexOf(authType) != -1) { if (serverSupportedTypes.indexOf(authType) != -1) {
this._sock.send([0, 0, 0, clientSupportedTypes[authType]]); this._sock.send([0, 0, 0, clientSupportedTypes[authType]]);
Log.Debug("Selected authentication type: " + authType);
switch (authType) { switch (authType) {
case 'STDVNOAUTH__': // no auth case 'STDVNOAUTH__': // no auth