fix: Default strategy event user (#5470)

Unleash · Nov 28, 2023 · 05614ed · 05614ed · vercel · Nov 28, 2023
1 parent 74e8d11
commit 05614ed
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/lib/routes/admin-api/project/environments.ts b/src/lib/routes/admin-api/project/environments.ts
@@ -162,7 +162,10 @@ export default class EnvironmentsController extends Controller {
     }
 
     async updateDefaultStrategyForProjectEnvironment(
-        req: Request<IProjectEnvironmentParams, CreateFeatureStrategySchema>,
+        req: IAuthRequest<
+            IProjectEnvironmentParams,
+            CreateFeatureStrategySchema
+        >,
         res: Response<CreateFeatureStrategySchema>,
     ): Promise<void> {
         const { projectId, environment } = req.params;
@@ -172,6 +175,7 @@ export default class EnvironmentsController extends Controller {
             environment,
             projectId,
             strategy,
+            extractUsername(req),
         );
 
         this.openApiService.respondWithValidation(

diff --git a/src/lib/services/environment-service.ts b/src/lib/services/environment-service.ts
@@ -131,7 +131,7 @@ export default class EnvironmentService {
         environment: string,
         projectId: string,
         strategy: CreateFeatureStrategySchema,
-        username: string = 'unknown',
+        username: string,
     ): Promise<CreateFeatureStrategySchema> {
         if (strategy.name !== 'flexibleRollout') {
             throw new BadDataError(