Filter some error reports out

Author: mustard-mh

src/local-ssh/proxy.ts

@@ -42,6 +42,10 @@ function getClientOptions(): ClientOptions {
 }
 
 type FailedToProxyCode = 'SSH.AuthenticationFailed' | 'TUNNEL.AuthenticateSSHKeyFailed' | 'NoRunningInstance' | 'FailedToGetAuthInfo';
+
+// IgnoredFailedCodes contains the failreCode that don't need to send error report
+const IgnoredFailedCodes: FailedToProxyCode[] = ['NoRunningInstance'];
+
 class FailedToProxyError extends Error {
     constructor(public readonly failureCode: FailedToProxyCode, originError?: Error) {
         const msg = 'Failed to proxy connection: ' + failureCode;
@@ -135,12 +139,16 @@ class WebSocketSSHProxy {
                     pipePromise = session.pipe(pipeSession);
                     return {};
                 }).catch(async err => {
+                    let sendErrorReport = true
                     if (err instanceof FailedToProxyError) {
                         this.flow.failureCode = err.failureCode;
+                        if (IgnoredFailedCodes.includes(err.failureCode)) {
+                            sendErrorReport = false
+                        }
                     }
                     await Promise.allSettled([
                         this.sendUserStatusFlow('failed'),
-                        this.sendErrorReport(this.flow, err, 'failed to authenticate proxy with username: ' + e.username ?? '')
+                        sendErrorReport ? this.sendErrorReport(this.flow, err, 'failed to authenticate proxy with username: ' + e.username ?? '') : undefined,
                     ]);
                     this.logService.error(err, 'failed to authenticate proxy with username: ' + e.username ?? '');
                     await session.close(SshDisconnectReason.byApplication, err.toString(), err instanceof Error ? err : undefined);

src/services/localSSHService.ts

@@ -26,6 +26,9 @@ export interface ILocalSSHService {
 
 type FailedToInitializeCode = 'Unknown' | 'LockFailed' | string;
 
+// IgnoredFailedCodes contains the codes that don't need to send error report
+const IgnoredFailedCodes: FailedToInitializeCode[] = ['ENOSPC'];
+
 export class LocalSSHService extends Disposable implements ILocalSSHService {
     public isSupportLocalSSH: boolean = false;
     public initialized!: Promise<void>;
@@ -74,14 +77,18 @@ export class LocalSSHService extends Disposable implements ILocalSSHService {
             });
         } catch (e) {
             this.logService.error(e, 'failed to initialize');
+            let sendErrorReport = true;
             failureCode = 'Unknown';
             if (e?.code) {
                 failureCode = e.code;
+                sendErrorReport = !IgnoredFailedCodes.includes(e.code)
             }
             if (e?.message) {
                 e.message = `Failed to initialize: ${e.message}`;
             }
-            this.telemetryService.sendTelemetryException(e, { gitpodHost: this.hostService.gitpodHost, useLocalAPP });
+            if (sendErrorReport) {
+                this.telemetryService.sendTelemetryException(e, { gitpodHost: this.hostService.gitpodHost, useLocalAPP });
+            }
             this.isSupportLocalSSH = false;
         }
         const flowData = this.flow ? this.flow : { gitpodHost: this.hostService.gitpodHost, userId: this.sessionService.safeGetUserId() };