[server] Inject process.env.HOST_URL into feature flag attributes

Author: geropl

components/gitpod-protocol/src/experiments/configcat-server.ts

@@ -54,6 +54,6 @@ export function getExperimentsClientForBackend(): Client {
         baseUrl: process.env.CONFIGCAT_BASE_URL,
     });
 
-    client = new ConfigCatClient(configCatClient);
+    client = new ConfigCatClient(configCatClient, process.env.HOST_URL);
     return client;
 }

components/gitpod-protocol/src/experiments/configcat.ts

@@ -17,14 +17,17 @@ export const BILLING_TIER_ATTRIBUTE = "billing_tier";
 export const GITPOD_HOST = "gitpod_host";
 
 export class ConfigCatClient implements Client {
-    private client: IConfigCatClient;
-
-    constructor(cc: IConfigCatClient) {
-        this.client = cc;
-    }
+    constructor(private readonly client: IConfigCatClient, private readonly gitpodHost?: string) {}
 
     getValueAsync<T>(experimentName: string, defaultValue: T, attributes: Attributes): Promise<T> {
-        return this.client.getValueAsync(experimentName, defaultValue, attributesToUser(attributes));
+        return this.client.getValueAsync(
+            experimentName,
+            defaultValue,
+            attributesToUser({
+                gitpodHost: this.gitpodHost,
+                ...attributes,
+            }),
+        );
     }
 
     dispose(): void {

components/server/src/container-module.ts

@@ -7,10 +7,6 @@
 import { ContainerModule } from "inversify";
 
 import { IAnalyticsWriter } from "@gitpod/gitpod-protocol/lib/analytics";
-import {
-    ConfigCatClientFactory,
-    getExperimentsClientForBackend,
-} from "@gitpod/gitpod-protocol/lib/experiments/configcat-server";
 import { GitpodFileParser } from "@gitpod/gitpod-protocol/lib/gitpod-file-parser";
 import { PrometheusClientCallMetrics } from "@gitpod/gitpod-protocol/lib/messaging/client-call-metrics";
 import { newAnalyticsWriterFromEnv } from "@gitpod/gitpod-protocol/lib/util/analytics";
@@ -296,12 +292,6 @@ export const productionContainerModule = new ContainerModule(
             })
             .inSingletonScope();
 
-        bind(ConfigCatClientFactory)
-            .toDynamicValue((ctx) => {
-                return () => getExperimentsClientForBackend();
-            })
-            .inSingletonScope();
-
         bind(VerificationService).toSelf().inSingletonScope();
 
         bind(UsageService).toSelf().inSingletonScope();

components/server/src/workspace/gitpod-server-impl.ts

@@ -141,10 +141,7 @@ import { IDEService } from "../ide-service";
 import { AttributionId } from "@gitpod/gitpod-protocol/lib/attribution";
 import { CostCenterJSON } from "@gitpod/gitpod-protocol/lib/usage";
 import { createCookielessId, maskIp } from "../analytics";
-import {
-    ConfigCatClientFactory,
-    getExperimentsClientForBackend,
-} from "@gitpod/gitpod-protocol/lib/experiments/configcat-server";
+import { getExperimentsClientForBackend } from "@gitpod/gitpod-protocol/lib/experiments/configcat-server";
 import { increaseDashboardErrorBoundaryCounter } from "../prometheus-metrics";
 import { LinkedInService } from "../linkedin-service";
 import { SnapshotService, WaitForSnapshotOptions } from "./snapshot-service";
@@ -235,8 +232,6 @@ export class GitpodServerImpl implements GitpodServerWithTracing, Disposable {
         @inject(VerificationService) private readonly verificationService: VerificationService,
         @inject(EntitlementService) private readonly entitlementService: EntitlementService,
 
-        @inject(ConfigCatClientFactory) private readonly configCatClientFactory: ConfigCatClientFactory,
-
         @inject(Authorizer) private readonly auth: Authorizer,
 
         @inject(BillingModes) private readonly billingModes: BillingModes,
@@ -672,7 +667,7 @@ export class GitpodServerImpl implements GitpodServerWithTracing, Disposable {
         const user = await this.checkUser("sendPhoneNumberVerificationToken");
 
         // Check if verify via call is enabled
-        const phoneVerificationByCall = await this.configCatClientFactory().getValueAsync(
+        const phoneVerificationByCall = await getExperimentsClientForBackend().getValueAsync(
             "phoneVerificationByCall",
             false,
             {
@@ -3178,7 +3173,7 @@ export class GitpodServerImpl implements GitpodServerWithTracing, Disposable {
 
     private async guardWithFeatureFlag(flagName: string, user: User, teamId: string) {
         // Guard method w/ a feature flag check
-        const isEnabled = await this.configCatClientFactory().getValueAsync(flagName, false, {
+        const isEnabled = await getExperimentsClientForBackend().getValueAsync(flagName, false, {
             user: user,
             teamId,
         });