admin/twenty
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

fix(sso|auth): fix sso signinup (#9651)

Browse Source 
Renamed `user` to `payload` for better context clarity and updated
related references. Adjusted the login token generation to use
`workspace.id`, improving readability and maintainability of the code.
This commit is contained in:
Antoine Moreaux
2025-01-15 19:27:05 +01:00
committed by GitHub
parent f722a2d619
commit 789ff30dc7
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
packages/twenty-server/src/engine/core-modules/auth/controllers/sso-auth.controller.ts
View File

@ -137,7 +137,7 @@ export class SSOAuthController {
} }
private async generateLoginToken( private async generateLoginToken(
user: { email: string } & Record<string, string>, payload: { email: string } & Record<string, string>,
identityProvider: WorkspaceSSOIdentityProvider, identityProvider: WorkspaceSSOIdentityProvider,
) { ) {
if (!identityProvider) { if (!identityProvider) {
@ -149,16 +149,16 @@ export class SSOAuthController {
const existingUser = await this.userRepository.findOne({ const existingUser = await this.userRepository.findOne({
where: { where: {
email: user.email, email: payload.email,
}, },
}); });
const { userData } = this.authService.formatUserDataPayload( const { userData } = this.authService.formatUserDataPayload(
user, payload,
existingUser, existingUser,
); );
await this.authService.signInUp({ const { workspace, user } = await this.authService.signInUp({
userData, userData,
workspace: identityProvider.workspace, workspace: identityProvider.workspace,
authParams: { authParams: {
@ -170,7 +170,7 @@ export class SSOAuthController {
identityProvider, identityProvider,
loginToken: await this.loginTokenService.generateLoginToken( loginToken: await this.loginTokenService.generateLoginToken(
user.email, user.email,
identityProvider.workspace.id, workspace.id,
), ),
}; };
} }