Skip to content

Conversation

@longsizhuo
Copy link
Member

得力于 Next.js Vercel Auth, Github OAth Apps, NEON Database,我们拥有了完全免费的登录系统

文档

@vercel
Copy link

vercel bot commented Sep 21, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
involutionhell-github-io Error Error Sep 21, 2025 7:50pm
yuling_web Error Error Sep 21, 2025 7:50pm

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codex Review: Here are some suggestions.

Reply with @codex fix comments to fix any unresolved comments.

About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you open a pull request for review, mark a draft as ready, or comment "@codex review". If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex fix this CI failure" or "@codex address that feedback".

auth.config.ts Outdated
Comment on lines 39 to 48
async signIn({ user, account, profile }) {
console.log("[auth] signIn payload", { user, account, profile });
return true;
},
async session({ session, token }) {
console.log("[auth] session payload", { session, token });
return session;
},
async jwt({ token, user, account, profile }) {
console.log("[auth] jwt payload", { token, user, account, profile });

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[P1] Remove sensitive payload logging in auth callbacks

The signIn, session, and jwt callbacks log their entire payloads (console.log("[auth] …", { user, account, profile, token })). In production these objects contain OAuth access tokens and user data, so the change will leak credentials and personal information into application logs. Consider gating the logs behind a development flag or removing them before release.

Useful? React with 👍 / 👎.

@longsizhuo longsizhuo merged commit db8758e into main Sep 21, 2025
6 of 8 checks passed
@longsizhuo longsizhuo deleted the credient branch October 8, 2025 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants