refactor getUserIdByGithubId を抽象化

saitamau-maximum · Nov 10, 2024 · 7830a3c · 7830a3c
1 parent e2aeed7
commit 7830a3c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/webapp/api/cb.ts b/webapp/api/cb.ts
@@ -119,7 +119,10 @@ app.get(
     }
 
     // すでになければ DB にユーザー情報を格納
-    const oauthConnInfo = await c.var.idpClient.getUserIdByGithubId(user.id)
+    const oauthConnInfo = await c.var.idpClient.getUserIdByOauthId(
+      1,
+      String(user.id),
+    )
     if (!oauthConnInfo) {
       const uuid = crypto.randomUUID().replaceAll('-', '')
       // とりあえず仮情報で埋める

diff --git a/webapp/repository/idp.ts b/webapp/repository/idp.ts
@@ -43,11 +43,12 @@ export class IdpRepository {
     return await this.getStmt.userById.bind(id).first()
   }
 
-  async getUserIdByGithubId(
-    githubId: number,
+  async getUserIdByOauthId(
+    oauthProviderId: number,
+    oauthUserId: string,
   ): Promise<{ user_id: string } | null> {
     return await this.getStmt.userIdByOauthId
-      .bind(1, githubId.toString())
+      .bind(oauthProviderId, oauthUserId)
       .first()
   }