fix(auth): map oauth login to correct user id instead of account id (#3256)

This commit is contained in:
Rakshit Kaintura
2026-07-27 13:38:30 +02:00
committed by GitHub
parent 0eb9ce012e
commit 34c03b1f73
2 changed files with 4 additions and 2 deletions
+3 -1
View File
@@ -75,7 +75,7 @@ describe("createProfileMapper", () => {
expect(dbMock.update).toHaveBeenCalledTimes(1);
expect(dbMock.updateSet).toHaveBeenCalledWith({ email: "legacy.user@example.com" });
expect(result).toEqual({
id: "legacy-user-id",
id: "user-1",
name: "Legacy User",
email: "legacy.user@example.com",
image: "https://example.com/new.png",
@@ -153,6 +153,7 @@ describe("createProfileMapper", () => {
expect(dbMock.select).toHaveBeenCalledTimes(2);
expect(result).toEqual({
id: "user-1",
name: "GitHub User",
email: "github.user@example.com",
image: "https://example.com/new.png",
@@ -189,6 +190,7 @@ describe("createProfileMapper", () => {
expect(dbMock.update).toHaveBeenCalledTimes(1);
expect(dbMock.updateSet).toHaveBeenCalledWith({ email: "legacy.user@example.com" });
expect(result).toEqual({
id: "user-1",
name: "Legacy User",
email: "legacy.user@example.com",
image: "https://example.com/new.png",
+1 -1
View File
@@ -186,7 +186,7 @@ export function createProfileMapper<TProfile extends OAuthProfile>({
await normalizeExistingUserEmail(existingUser.id, existingUser.email, existingEmail);
return {
...(existingUser.accountId ? { id: existingUser.accountId } : {}),
id: existingUser.id,
name: existingUser.name,
email: existingEmail,
image: image ?? existingUser.image,