Skip to content

Commit 24b965d

Browse files
fix(oauth): remove internal error message from redirect URL
Remove the 'message' query parameter that exposed raw error.message content (potentially database errors, function names, or internal details) in the redirect URL. The generic 'CALLBACK_FAILED' error code is sufficient for the frontend, and detailed errors are still logged server-side. Co-Authored-By: Claude <noreply@anthropic.com>
1 parent cefc376 commit 24b965d

1 file changed

Lines changed: 0 additions & 1 deletion

File tree

graphql/server/src/middleware/oauth.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -567,7 +567,6 @@ export function createOAuthRoutes(opts: ConstructiveOptions): Router {
567567

568568
const errorUrl = new URL(errorRedirectPath, getBaseUrl(req));
569569
errorUrl.searchParams.set('error', 'CALLBACK_FAILED');
570-
errorUrl.searchParams.set('message', error.message || 'Unknown error');
571570
errorUrl.searchParams.set('provider', provider);
572571
return res.redirect(errorUrl.toString());
573572
}

0 commit comments

Comments
 (0)