fix: redirect_uri 指定しないと invalid auth_token になる (#49)

saitamau-maximum · Dec 16, 2024 · eb114f3 · eb114f3
1 parent 586ad80
commit eb114f3
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 8 deletions.
diff --git a/webapp/api/oauth/_templates/authorize.ts b/webapp/api/oauth/_templates/authorize.ts
@@ -7,7 +7,10 @@ interface AuthorizeProps {
   scopes: { name: string; description: string | null }[]
   oauthFields: {
     clientId: string
-    redirectUri: string
+    // client が指定してきた redirect_uri
+    redirectUri?: string
+    // client が指定してきた redirect_uri または DB に保存されている callback
+    redirectTo: string
     state?: string
     scope?: string
     token: string
@@ -66,11 +69,13 @@ export const _Authorize = ({
       </div>
       <form method="POST" action="/oauth/callback" class="space-y-4">
         <input type="hidden" name="client_id" value="${oauthFields.clientId}" />
-        <input
-          type="hidden"
-          name="redirect_uri"
-          value="${oauthFields.redirectUri}"
-        />
+        ${oauthFields.redirectUri
+          ? html`<input
+              type="hidden"
+              name="redirect_uri"
+              value="${oauthFields.redirectUri}"
+            />`
+          : ''}
         ${oauthFields.state
           ? html`<input
               type="hidden"
@@ -106,7 +111,7 @@ export const _Authorize = ({
       </form>
     </div>
     <p class="text-sm text-gray-600 text-center">
-      ${new URL(oauthFields.redirectUri).origin} へリダイレクトします。
+      ${new URL(oauthFields.redirectTo).origin} へリダイレクトします。
       このアドレスが意図しているものか確認してください。
     </p>
   </div>

diff --git a/webapp/api/oauth/authorize.ts b/webapp/api/oauth/authorize.ts
@@ -225,7 +225,8 @@ app.get(
         })),
         oauthFields: {
           clientId,
-          redirectUri: redirectTo,
+          redirectUri,
+          redirectTo,
           state,
           scope,
           token,