Avoid logging csr

Summary: I unintentionally added logging of csr certificates in my previous diff. This diff makes sure we're not logging them anymore.

Reviewed By: mweststrate

Differential Revision: D31609709

fbshipit-source-id: e43b348dbd62653f8a6e9089930c2a3699b29a12

desktop/flipper-server-core/src/comms/ServerController.tsx

@@ -25,6 +25,7 @@ import invariant from 'invariant';
 import DummyDevice from '../devices/DummyDevice';
 import {
   appNameWithUpdateHint,
+  cloneClientQuerySafeForLogging,
   transformCertificateExchangeMediumToType,
 } from './Utilities';
 import ServerAdapter, {
@@ -192,7 +193,7 @@ class ServerController extends EventEmitter implements ServerEventsListener {
     const transformedMedium = transformCertificateExchangeMediumToType(medium);
     console.info(
       `[conn] Connection established: ${app} on ${device_id}. Medium ${medium}. CSR: ${csr_path}`,
-      clientQuery,
+      cloneClientQuerySafeForLogging(clientQuery),
     );
     return this.addConnection(
       clientConnection,
@@ -356,7 +357,6 @@ class ServerController extends EventEmitter implements ServerEventsListener {
       console.info(
         `[conn] Detected ${app_name} on ${query.device_id} in certificate`,
         query,
-        csrQuery,
       );
     }
 
@@ -372,7 +372,6 @@ class ServerController extends EventEmitter implements ServerEventsListener {
     console.info(
       `[conn] Matching device for ${query.app} on ${query.device_id}...`,
       query,
-      csrQuery,
     );
 
     const client: ClientDescription = {
@@ -388,7 +387,6 @@ class ServerController extends EventEmitter implements ServerEventsListener {
     console.info(
       `[conn] Initializing client ${query.app} on ${query.device_id}...`,
       query,
-      csrQuery,
     );
 
     connection.subscribeToEvents((status: ConnectionStatus) => {
@@ -400,12 +398,7 @@ class ServerController extends EventEmitter implements ServerEventsListener {
       }
     });
 
-    console.debug(
+    console.debug(`[conn] Device client initialized: ${id}.`, 'server', query);
-      `[conn] Device client initialized: ${id}.`,
-      'server',
-      query,
-      csrQuery,
-    );
 
     /* If a device gets disconnected without being cleaned up properly,
      * Flipper won't be aware until it attempts to reconnect.

desktop/flipper-server-core/src/comms/ServerWebSocket.tsx

@@ -24,6 +24,7 @@ import {
   ClientQuery,
   DeviceOS,
 } from 'flipper-common';
+import {cloneClientQuerySafeForLogging} from './Utilities';
 
 /**
  * WebSocket-based server.
@@ -95,7 +96,7 @@ class ServerWebSocket extends ServerWebSocketBase {
     }
     console.info(
       `[conn] Secure websocket connection attempt: ${clientQuery.app} on ${clientQuery.device_id}. Medium ${clientQuery.medium}. CSR: ${clientQuery.csr_path}`,
-      clientQuery,
+      cloneClientQuerySafeForLogging(clientQuery),
     );
     this.listener.onSecureConnectionAttempt(clientQuery);
 

desktop/flipper-server-core/src/comms/Utilities.tsx

@@ -9,6 +9,7 @@
 
 import {ClientQuery} from 'flipper-common';
 import {CertificateExchangeMedium} from '../utils/CertificateProvider';
+import {SecureClientQuery} from './ServerAdapter';
 
 /**
  * Transforms the certificate exchange medium type as number to the
@@ -48,3 +49,7 @@ export function appNameWithUpdateHint(query: ClientQuery): string {
   }
   return query.app;
 }
+
+export function cloneClientQuerySafeForLogging(clientQuery: SecureClientQuery) {
+  return {...clientQuery, csr: !clientQuery.csr ? clientQuery.csr : '<hidden>'};
+}