From 12e36ad082733f3cde6d885ede22739a2900e2d3 Mon Sep 17 00:00:00 2001 From: bo0tzz Date: Sat, 18 Apr 2026 17:25:15 +0200 Subject: [PATCH] chore: remove deleteByEmail --- server/src/repositories/oauth-link-token.repository.ts | 4 ---- server/src/services/auth.service.spec.ts | 4 ---- server/src/services/auth.service.ts | 1 - 3 files changed, 9 deletions(-) diff --git a/server/src/repositories/oauth-link-token.repository.ts b/server/src/repositories/oauth-link-token.repository.ts index 737ca8c025..8dea55e7ac 100644 --- a/server/src/repositories/oauth-link-token.repository.ts +++ b/server/src/repositories/oauth-link-token.repository.ts @@ -22,10 +22,6 @@ export class OAuthLinkTokenRepository { .executeTakeFirst(); } - async deleteByEmail(userEmail: string) { - await this.db.deleteFrom('oauth_link_token').where('userEmail', '=', userEmail).execute(); - } - async cleanup() { const result = await this.db .deleteFrom('oauth_link_token') diff --git a/server/src/services/auth.service.spec.ts b/server/src/services/auth.service.spec.ts index d9e7abc763..963ece6c46 100644 --- a/server/src/services/auth.service.spec.ts +++ b/server/src/services/auth.service.spec.ts @@ -762,7 +762,6 @@ describe(AuthService.name, () => { mocks.systemMetadata.get.mockResolvedValue(systemConfigStub.oauthEnabled); mocks.oauth.getProfileAndOAuthSid.mockResolvedValue({ profile, sid: 'idp-sid-789' }); mocks.user.getByEmail.mockResolvedValue(user); - mocks.oauthLinkToken.deleteByEmail.mockResolvedValue(); mocks.oauthLinkToken.create.mockResolvedValue({} as any); await expect( @@ -775,7 +774,6 @@ describe(AuthService.name, () => { expect(mocks.user.getByEmail).toHaveBeenCalledTimes(1); expect(mocks.user.update).not.toHaveBeenCalled(); - expect(mocks.oauthLinkToken.deleteByEmail).toHaveBeenCalledTimes(1); expect(mocks.oauthLinkToken.create).toHaveBeenCalledWith( expect.objectContaining({ oauthSub: profile.sub, oauthSid: 'idp-sid-789' }), ); @@ -788,7 +786,6 @@ describe(AuthService.name, () => { mocks.systemMetadata.get.mockResolvedValue(systemConfigStub.oauthEnabled); mocks.oauth.getProfileAndOAuthSid.mockResolvedValue({ profile }); mocks.user.getByEmail.mockResolvedValue(user); - mocks.oauthLinkToken.deleteByEmail.mockResolvedValue(); mocks.oauthLinkToken.create.mockResolvedValue({} as any); await expect( @@ -810,7 +807,6 @@ describe(AuthService.name, () => { mocks.oauth.getProfileAndOAuthSid.mockResolvedValue({ profile: OAuthProfileFactory.create() }); mocks.user.getByEmail.mockResolvedValueOnce(user); mocks.user.getAdmin.mockResolvedValue(UserFactory.create({ isAdmin: true })); - mocks.oauthLinkToken.deleteByEmail.mockResolvedValue(); mocks.oauthLinkToken.create.mockResolvedValue({} as any); await expect( diff --git a/server/src/services/auth.service.ts b/server/src/services/auth.service.ts index d5048f7065..2eb895dbbc 100644 --- a/server/src/services/auth.service.ts +++ b/server/src/services/auth.service.ts @@ -350,7 +350,6 @@ export class AuthService extends BaseService { if (!user && normalizedEmail) { const emailUser = await this.userRepository.getByEmail(normalizedEmail); if (emailUser) { - await this.oauthLinkTokenRepository.deleteByEmail(emailUser.email); const plainToken = this.cryptoRepository.randomBytesAsText(32); const hashedToken = this.cryptoRepository.hashSha256(plainToken); await this.oauthLinkTokenRepository.create({