fix(cli): password reset should trigger internal and external hooks (#…

…4066) Ticket: N8N-4586
n8n-io · Sep 9, 2022 · 12507d3 · 12507d3
1 parent eebc51e
commit 12507d3
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/packages/cli/src/UserManagement/routes/passwordReset.ts b/packages/cli/src/UserManagement/routes/passwordReset.ts
@@ -215,6 +215,13 @@ export function passwordResetNamespace(this: N8nApp): void {
 			Logger.info('User password updated successfully', { userId });
 
 			await issueCookie(res, user);
+
+			void InternalHooksManager.getInstance().onUserUpdate({
+				user_id: userId,
+				fields_changed: ['password'],
+			});
+
+			await this.externalHooks.run('user.password.update', [user.email, password]);
 		}),
 	);
 }
diff --git a/packages/cli/test/integration/shared/utils.ts b/packages/cli/test/integration/shared/utils.ts
@@ -101,7 +101,8 @@ export async function initTestServer({
 	if (
 		endpointGroups.includes('credentials') ||
 		endpointGroups.includes('me') ||
-		endpointGroups.includes('users')
+		endpointGroups.includes('users') ||
+		endpointGroups.includes('passwordReset')
 	) {
 		testServer.externalHooks = ExternalHooks();
 	}