Also delete any LinkedIn profile when deleting a user account

Author: jankeromnes

components/gitpod-db/src/linked-in-profile-db.ts

@@ -9,4 +9,5 @@ import { LinkedInProfile } from "@gitpod/gitpod-protocol";
 export const LinkedInProfileDB = Symbol("LinkedInProfileDB");
 export interface LinkedInProfileDB {
     storeProfile(userId: string, profile: LinkedInProfile): Promise<void>;
+    deleteProfile(userId: string): Promise<void>;
 }

components/gitpod-db/src/typeorm/linked-in-profile-db-impl.ts

@@ -41,4 +41,9 @@ export class LinkedInProfileDBImpl implements LinkedInProfileDB {
             creationTime: existingProfile?.creationTime || new Date().toISOString(),
         });
     }
+
+    public async deleteProfile(userId: string): Promise<void> {
+        const repo = await this.getRepo();
+        await repo.delete({ userId });
+    }
 }

components/server/src/linkedin-service.ts

@@ -26,6 +26,10 @@ export class LinkedInService {
         return profile;
     }
 
+    async deleteLinkedInProfile(userId: string): Promise<void> {
+        await this.linkedInProfileDB.deleteProfile(userId);
+    }
+
     private async getAccessToken(code: string) {
         const { clientId, clientSecret } = this.config.linkedInSecrets || {};
         if (!clientId || !clientSecret) {

components/server/src/user/user-deletion-service.ts

@@ -17,6 +17,7 @@ import { IAnalyticsWriter } from "@gitpod/gitpod-protocol/lib/analytics";
 import { Config } from "../config";
 import { StripeService } from "../../ee/src/user/stripe-service";
 import { BillingModes } from "../../ee/src/billing/billing-mode";
+import { LinkedInService } from "../linkedin-service";
 
 @injectable()
 export class UserDeletionService {
@@ -34,6 +35,7 @@ export class UserDeletionService {
     @inject(IAnalyticsWriter) protected readonly analytics: IAnalyticsWriter;
     @inject(StripeService) protected readonly stripeService: StripeService;
     @inject(BillingModes) protected readonly billingMode: BillingModes;
+    @inject(LinkedInService) protected readonly linkedInService: LinkedInService;
 
     /**
      * This method deletes a User logically. The contract here is that after running this method without receiving an
@@ -93,6 +95,8 @@ export class UserDeletionService {
             this.deleteTeamMemberships(id),
             // User projects
             this.deleteUserProjects(id),
+            // LinkedIn profile
+            this.deleteLinkedInProfile(id),
         ]);
 
         // Track the deletion Event for Analytics Purposes
@@ -220,6 +224,14 @@ export class UserDeletionService {
         await Promise.all(userProjects.map((project) => this.projectDb.markDeleted(project.id)));
     }
 
+    protected async deleteLinkedInProfile(userId: string) {
+        try {
+            await this.linkedInService.deleteLinkedInProfile(userId);
+        } catch (error) {
+            log.error({ userId: userId }, "Failed to delete user's LinkedIn profile.", error);
+        }
+    }
+
     anonymizeWorkspace(ws: Workspace) {
         ws.context.title = "deleted-title";
         ws.context.normalizedContextURL = "deleted-normalizedContextURL";