chore: remove manual anonymization of outgoing project owners (#8252)

This change removes the flag used to anonymize project owners on the
way out. It was an issue in demo when we'd forgotten to configure the
email encryption. However, this issue has been resolved and we can
remove this check now.

src/lib/features/project/project-owners-read-model.test.ts

@@ -361,25 +361,4 @@ describe('integration tests', () => {
             { name: projectIdB, owners: [{ ownerType: 'user' }] },
         ]);
     });
-
-    test('anonymizes emails when asked to', async () => {
-        const projectId = randomId();
-        await db.stores.projectStore.create({ id: projectId, name: projectId });
-
-        await db.stores.accessStore.addUserToRole(
-            owner.id,
-            ownerRoleId,
-            projectId,
-        );
-
-        const owners = await readModel.getAllProjectOwners(true);
-        expect(owners).toMatchObject({
-            [projectId]: [
-                {
-                    name: 'Owner Name',
-                    email: expect.stringMatching(/@unleash.run$/),
-                },
-            ],
-        });
-    });
 });

src/lib/features/project/project-owners-read-model.ts

@@ -1,6 +1,6 @@
 import type { Db } from '../../db/db';
 import { RoleName } from '../../types';
-import { anonymise, generateImageUrl } from '../../util';
+import { generateImageUrl } from '../../util';
 import type {
     GroupProjectOwner,
     IProjectOwnersReadModel,
@@ -35,7 +35,6 @@ export class ProjectOwnersReadModel implements IProjectOwnersReadModel {
 
     private async getAllProjectUsersByRole(
         roleId: number,
-        anonymizeProjectOwners: boolean = false,
     ): Promise<Record<string, UserProjectOwner[]>> {
         const usersResult = await this.db
             .select(
@@ -54,20 +53,13 @@ export class ProjectOwnersReadModel implements IProjectOwnersReadModel {
             .join(`${T.USERS} as user`, 'ru.user_id', 'user.id');
         const usersDict: Record<string, UserProjectOwner[]> = {};
 
-        const processSensitiveData = anonymizeProjectOwners
-            ? anonymise
-            : (x: string) => x;
-
         usersResult.forEach((user) => {
             const project = user.project as string;
 
             const data: UserProjectOwner = {
                 ownerType: 'user',
-                name:
-                    user?.name ||
-                    user?.username ||
-                    processSensitiveData(user?.email),
-                email: processSensitiveData(user?.email),
+                name: user?.name || user?.username || user?.email,
+                email: user?.email,
                 imageUrl: generateImageUrl(user),
             };
 
@@ -112,16 +104,11 @@ export class ProjectOwnersReadModel implements IProjectOwnersReadModel {
         return groupsDict;
     }
 
-    async getAllProjectOwners(
-        anonymizeProjectOwners: boolean = false,
-    ): Promise<ProjectOwnersDictionary> {
+    async getAllProjectOwners(): Promise<ProjectOwnersDictionary> {
         const ownerRole = await this.db(T.ROLES)
             .where({ name: RoleName.OWNER })
             .first();
-        const usersDict = await this.getAllProjectUsersByRole(
-            ownerRole.id,
-            anonymizeProjectOwners,
-        );
+        const usersDict = await this.getAllProjectUsersByRole(ownerRole.id);
         const groupsDict = await this.getAllProjectGroupsByRole(ownerRole.id);
 
         const dict: Record<
@@ -142,9 +129,8 @@ export class ProjectOwnersReadModel implements IProjectOwnersReadModel {
 
     async addOwners<T extends { id: string }>(
         projects: T[],
-        anonymizeProjectOwners: boolean = false,
     ): Promise<WithProjectOwners<T>> {
-        const owners = await this.getAllProjectOwners(anonymizeProjectOwners);
+        const owners = await this.getAllProjectOwners();
 
         return ProjectOwnersReadModel.addOwnerData(projects, owners);
     }

src/lib/features/project/project-owners-read-model.type.ts

@@ -28,6 +28,5 @@ export type WithProjectOwners<T extends { id: string }> = (T & {
 export interface IProjectOwnersReadModel {
     addOwners<T extends { id: string }>(
         projects: T[],
-        anonymizeProjectOwners?: boolean,
     ): Promise<WithProjectOwners<T>>;
 }

src/lib/features/project/project-service.ts

@@ -259,13 +259,7 @@ export default class ProjectService {
     async addOwnersToProjects(
         projects: ProjectForUi[],
     ): Promise<ProjectForUi[]> {
-        const anonymizeProjectOwners = this.flagResolver.isEnabled(
-            'anonymizeProjectOwners',
-        );
-        return this.projectOwnersReadModel.addOwners(
-            projects,
-            anonymizeProjectOwners,
-        );
+        return this.projectOwnersReadModel.addOwners(projects);
     }
 
     async getProject(id: string): Promise<IProject> {

src/lib/types/experimental.ts

@@ -52,7 +52,6 @@ export type IFlagKey =
     | 'manyStrategiesPagination'
     | 'enableLegacyVariants'
     | 'navigationSidebar'
-    | 'anonymizeProjectOwners'
     | 'extendedMetrics'
     | 'removeUnsafeInlineStyleSrc'
     | 'originMiddleware'
@@ -262,10 +261,6 @@ const flags: IFlags = {
         process.env.UNLEASH_EXPERIMENTAL_SIDEBAR_NAVIGATION,
         true,
     ),
-    anonymizeProjectOwners: parseEnvVarBoolean(
-        process.env.UNLEASH_EXPERIMENTAL_ANONYMIZE_PROJECT_OWNERS,
-        false,
-    ),
     extendedMetrics: parseEnvVarBoolean(
         process.env.UNLEASH_EXPERIMENTAL_EXTENDED_METRICS,
         false,