fix lint

docs/docs/getting-started/client.md

@@ -163,6 +163,10 @@ NextAuth.js provides a `getSession()` helper which should be called **client sid
 
 On the server side, **this is still available to use**, however, we recommend using `getServerSession` going forward. The idea behind this is to avoid an additional unnecessary `fetch` call on the server side. For more information, please check out [this issue](https://github.com/nextauthjs/next-auth/issues/1535).
 
+:::note
+The `getServerSession` only has the prefix `unstable_` at the moment, because the API may change in the future. There are no known bugs at the moment and it is safe to use. If you discover any issues, please do report them as a [GitHub Issue](https://github.com/nextauthjs/next-auth/issues) and we will patch them as soon as possible.
+:::
+
 This helper is helpful in case you want to read the session outside of the context of React.
 
 When called, `getSession()` will send a request to `/api/auth/session` and returns a promise with a [session object](https://github.com/nextauthjs/next-auth/blob/main/packages/next-auth/src/core/types.ts#L407-L425), or `null` if no session exists.

packages/next-auth/package.json

@@ -1,6 +1,6 @@
 {
   "name": "next-auth",
-  "version": "4.19.1",
+  "version": "4.18.10",
   "description": "Authentication for Next.js",
   "homepage": "https://next-auth.js.org",
   "repository": "https://github.com/nextauthjs/next-auth.git",

packages/next-auth/src/core/lib/callback-handler.ts

@@ -113,7 +113,7 @@ export default async function callbackHandler(params: {
     session = useJwtSession
       ? {}
       : await createSession({
-          sessionToken: await generateSessionToken(),
+          sessionToken: generateSessionToken(),
           userId: user.id,
           expires: fromDate(options.session.maxAge),
         })
@@ -143,7 +143,7 @@ export default async function callbackHandler(params: {
       session = useJwtSession
         ? {}
         : await createSession({
-            sessionToken: await generateSessionToken(),
+            sessionToken: generateSessionToken(),
             userId: userByAccount.id,
             expires: fromDate(options.session.maxAge),
           })
@@ -181,11 +181,11 @@ export default async function callbackHandler(params: {
         ? await getUserByEmail(profile.email)
         : null
       if (userByEmail) {
-        const provider = options.provider as OAuthConfig<any>
+        const provider = options.provider as OAuthConfig<any>;
         if (provider?.allowDangerousEmailAccountLinking) {
-          // If you trust the oauth provider to correctly verify email addresses, you can opt-in to
+          // If you trust the oauth provider to correctly verify email addresses, you can opt-in to 
           // account linking even when the user is not signed-in.
-          user = userByEmail
+          user = userByEmail;
         } else {
           // We end up here when we don't have an account with the same [provider].id *BUT*
           // we do already have an account with the same email address as the one in the
@@ -216,7 +216,7 @@ export default async function callbackHandler(params: {
       session = useJwtSession
         ? {}
         : await createSession({
-            sessionToken: await generateSessionToken(),
+            sessionToken: generateSessionToken(),
             userId: user.id,
             expires: fromDate(options.session.maxAge),
           })

packages/next-auth/src/core/lib/oauth/authorization-url.ts

@@ -1,5 +1,5 @@
 import { openidClient } from "./client"
-import { oAuth1Client, oAuth1TokenStore } from "./client-legacy"
+import { oAuth1Client } from "./client-legacy"
 import { createState } from "./state-handler"
 import { createNonce } from "./nonce-handler"
 import { createPKCE } from "./pkce-handler"
@@ -44,7 +44,7 @@ export default async function getAuthorizationUrl({
       oauth_token_secret: tokens.oauth_token_secret,
       ...tokens.params,
     })}`
-    oAuth1TokenStore.set(tokens.oauth_token, tokens.oauth_token_secret)
+
     logger.debug("GET_AUTHORIZATION_URL", { url, provider })
     return { redirect: url }
   }

packages/next-auth/src/core/lib/oauth/callback.ts

@@ -1,6 +1,6 @@
 import { TokenSet } from "openid-client"
 import { openidClient } from "./client"
-import { oAuth1Client, oAuth1TokenStore } from "./client-legacy"
+import { oAuth1Client } from "./client-legacy"
 import { useState } from "./state-handler"
 import { usePKCECodeVerifier } from "./pkce-handler"
 import { useNonce } from "./nonce-handler"
@@ -42,7 +42,7 @@ export default async function oAuthCallback(params: {
       const { oauth_token, oauth_verifier } = query ?? {}
       const tokens = (await (client as any).getOAuthAccessToken(
         oauth_token,
-        oAuth1TokenStore.get(oauth_token),
+        null,
         oauth_verifier
       )) as TokenSet
       let profile: Profile = await (client as any).get(
@@ -63,8 +63,6 @@ export default async function oAuthCallback(params: {
     }
   }
 
-  if (query?.oauth_token) oAuth1TokenStore.delete(query.oauth_token)
-
   try {
     const client = await openidClient(options)
 

packages/next-auth/src/core/lib/oauth/client-legacy.ts

@@ -69,5 +69,3 @@ export function oAuth1Client(options: InternalOptions<"oauth">) {
   }
   return oauth1Client
 }
-
-export const oAuth1TokenStore = new Map()

packages/next-auth/src/core/pages/error.tsx

@@ -102,8 +102,8 @@ export default function ErrorPage(props: ErrorProps) {
             }}
           />
         )}
+        {theme?.logo && <img src={theme.logo} alt="Logo" className="logo" />}
         <div className="card">
-          {theme?.logo && <img src={theme.logo} alt="Logo" className="logo" />}
           <h1>{heading}</h1>
           <div className="message">{message}</div>
           {signin}

packages/next-auth/src/core/pages/signin.tsx

@@ -49,13 +49,6 @@ export default function SigninPage(props: SignInServerPageParams) {
     return false
   })
 
-  if (typeof document !== "undefined" && theme.buttonText) {
-    document.documentElement.style.setProperty(
-      "--button-text-color",
-      theme.buttonText
-    )
-  }
-
   if (typeof document !== "undefined" && theme.brandColor) {
     document.documentElement.style.setProperty(
       "--brand-color",
@@ -95,17 +88,7 @@ export default function SigninPage(props: SignInServerPageParams) {
           }}
         />
       )}
-      {theme.buttonText && (
-        <style
-          dangerouslySetInnerHTML={{
-            __html: `
-        :root {
-          --button-text-color: ${theme.buttonText}
-        }
-      `,
-          }}
-        />
-      )}
+      {theme.logo && <img src={theme.logo} alt="Logo" className="logo" />}
       <div className="card">
         {error && (
           <div className="error">
@@ -181,7 +164,7 @@ export default function SigninPage(props: SignInServerPageParams) {
                   placeholder="email@example.com"
                   required
                 />
-                <button id="submitButton" type="submit">Sign in with {provider.name}</button>
+                <button type="submit">Sign in with {provider.name}</button>
               </form>
             )}
             {provider.type === "credentials" && (

packages/next-auth/src/core/pages/signout.tsx

@@ -23,24 +23,13 @@ export default function SignoutPage(props: SignoutProps) {
           }}
         />
       )}
-      {theme.buttonText && (
-        <style
-          dangerouslySetInnerHTML={{
-            __html: `
-        :root {
-          --button-text-color: ${theme.buttonText}
-        }
-      `,
-          }}
-        />
-      )}
+      {theme.logo && <img src={theme.logo} alt="Logo" className="logo" />}
       <div className="card">
-        {theme.logo && <img src={theme.logo} alt="Logo" className="logo" />}
         <h1>Signout</h1>
         <p>Are you sure you want to sign out?</p>
         <form action={`${url}/signout`} method="POST">
           <input type="hidden" name="csrfToken" value={csrfToken} />
-          <button id="submitButton" type="submit">Sign out</button>
+          <button type="submit">Sign out</button>
         </form>
       </div>
     </div>

packages/next-auth/src/core/pages/verify-request.tsx

@@ -22,8 +22,8 @@ export default function VerifyRequestPage(props: VerifyRequestPageProps) {
           }}
         />
       )}
+      {theme.logo && <img src={theme.logo} alt="Logo" className="logo" />}
       <div className="card">
-        {theme.logo && <img src={theme.logo} alt="Logo" className="logo" />}
         <h1>Check your email</h1>
         <p>A sign in link has been sent to your email address.</p>
         <p>

packages/next-auth/src/core/types.ts

@@ -465,7 +465,7 @@ export interface SessionOptions {
    * However, you can specify your own custom string (such as CUID) to be used.
    * @default `randomUUID` or `randomBytes.toHex` depending on the Node.js version
    */
-  generateSessionToken: () => Awaitable<string>
+  generateSessionToken: () => string
 }
 
 export interface DefaultUser {

packages/next-auth/src/css/index.css

@@ -8,8 +8,7 @@
 
 .__next-auth-theme-auto,
 .__next-auth-theme-light {
-  --color-background: #ececec;
-  --color-background-card: #fff;
+  --color-background: #fff;
   --color-text: #000;
   --color-primary: #444;
   --color-control-border: #bbb;
@@ -19,8 +18,7 @@
 }
 
 .__next-auth-theme-dark {
-  --color-background: #161b22;
-  --color-background-card: #0d1117;
+  --color-background: #000;
   --color-text: #fff;
   --color-primary: #ccc;
   --color-control-border: #555;
@@ -31,8 +29,7 @@
 
 @media (prefers-color-scheme: dark) {
   .__next-auth-theme-auto {
-    --color-background: #161b22;
-    --color-background-card: #0d1117;
+    --color-background: #000;
     --color-text: #fff;
     --color-primary: #ccc;
     --color-control-border: #555;
@@ -81,9 +78,10 @@ input[type] {
   width: 100%;
   padding: 0.5rem 1rem;
   border: var(--border-width) solid var(--color-control-border);
-  background: var(--color-background-card);
+  background: var(--color-background);
   font-size: 1rem;
   border-radius: var(--border-radius);
+  box-shadow: inset 0 0.1rem 0.2rem rgba(0, 0, 0, 0.2);
   color: var(--color-text);
 
   &:focus {
@@ -109,39 +107,41 @@ a.button {
   }
 }
 
-button span {
-  flex-grow: 1;
-}
-
 button,
 a.button {
   margin: 0 0 0.75rem 0;
   padding: 0.75rem 1rem;
   color: var(--provider-color, var(--color-primary));
-  background-color: var(--provider-bg, var(--color-background-card));
+  background-color: var(--provider-bg, var(--color-background));
   font-size: 1.1rem;
   min-height: 62px;
   border-color: rgba(0, 0, 0, 0.1);
   border-radius: var(--border-radius);
   transition: all 0.1s ease-in-out;
+  box-shadow: #000 0px 0px 0px 0px, #000 0px 0px 0px 0px,
+    rgba(0, 0, 0, 0.2) 0px 10px 15px -3px, rgba(0, 0, 0, 0.1) 0px 4px 6px -4px;
   font-weight: 500;
   position: relative;
   display: flex;
   align-items: center;
   justify-content: center;
 
-  @media (max-width: 450px) {
-    font-size: 0.9rem;
+  &:has(img) {
+    justify-content: unset;
+    span {
+      flex-grow: 1;
+    }
   }
-
   &:hover {
     cursor: pointer;
   }
   &:active {
+    box-shadow: 0 0.15rem 0.3rem rgba(0, 0, 0, 0.15),
+      inset 0 0.1rem 0.2rem var(--color-background),
+      inset 0 -0.1rem 0.1rem rgba(0, 0, 0, 0.1);
     cursor: pointer;
   }
   #provider-logo {
-    width: 25px;
     display: block;
   }
   #provider-logo-dark {
@@ -149,23 +149,20 @@ a.button {
   }
 }
 
-#submitButton {
-  color: var(--button-text-color, var(--color-info-text));
-  background-color: var(--brand-color, var(--color-info));
-  width: 100%;
-}
-
 @media (prefers-color-scheme: dark) {
   button,
   a.button {
     color: var(--provider-dark-color, var(--color-primary));
     background-color: var(--provider-dark-bg, var(--color-background));
+    border: 1px solid #0d0d0d;
+    box-shadow: #000 0px 0px 0px 0px, #ccc 0px 0px 0px 0px,
+      rgba(255, 255, 255, 0.01) 0px 5px 5px -3px,
+      rgba(255, 255, 255, 0.05) 0px 4px 6px -4px;
   }
   #provider-logo {
     display: none !important;
   }
   #provider-logo-dark {
-    width: 25px;
     display: block !important;
   }
 }
@@ -192,6 +189,7 @@ a.site {
 
   > div {
     text-align: center;
+    padding: 0.5rem;
   }
 }
 
@@ -219,16 +217,16 @@ a.site {
     display: block;
     border: 0;
     border-top: 1px solid var(--color-seperator);
-    margin: 2rem auto 1rem auto;
+    margin: 1.5em auto 0 auto;
     overflow: visible;
 
     &::before {
       content: "or";
-      background: var(--color-background-card);
+      background: var(--color-background);
       color: #888;
       padding: 0 0.4rem;
       position: relative;
-      top: -0.7rem;
+      top: -0.6rem;
     }
   }
 
@@ -236,7 +234,7 @@ a.site {
     background: #f5f5f5;
     font-weight: 500;
     border-radius: 0.3rem;
-    background: var(--color-error);
+    background: var(--color-info);
 
     p {
       text-align: left;
@@ -262,26 +260,25 @@ a.site {
     max-width: 300px;
   }
 }
+.signout {
+  .message {
+    margin-bottom: 1.5rem;
+  }
+}
 
 .logo {
   display: inline-block;
-  max-width: 150px;
-  margin-top: 20px;
-  margin-bottom: 25px;
-  max-height: 70px;
+  margin-top: 100px;
+  max-width: 300px;
+  max-height: 150px;
 }
 
 .card {
-  @media screen and (min-width: 450px) {
-    width: 350px;
-  }
-  @media screen and (max-width: 450px) {
-    width: 200px;
-  }
-  margin: 20px 0 20px 0;
-  background-color: var(--color-background-card);
-  border-radius: 30px;
+  max-width: max-content;
+  border: 1px solid var(--color-control-border);
+  border-radius: 5px;
   padding: 20px 50px;
+  margin: 50px auto;
 
   .header {
     color: var(--color-primary);
@@ -289,5 +286,5 @@ a.site {
 }
 
 .section-header {
-  color: var(--color-text);
+  color: var(--brand-color, var(--color-text));
 }

packages/next-auth/src/next/index.ts

@@ -170,14 +170,9 @@ export async function getServerSession<
 let deprecatedWarningShown = false
 
 /** @deprecated renamed to `getServerSession` */
-export async function unstable_getServerSession<
-  O extends GetServerSessionOptions,
-  R = O["callbacks"] extends { session: (...args: any[]) => infer U }
-    ? U
-    : Session
->(
-  ...args: Parameters<typeof getServerSession<O, R>>
-): ReturnType<typeof getServerSession<O, R>> {
+export async function unstable_getServerSession(
+  ...args: Parameters<typeof getServerSession>
+): ReturnType<typeof getServerSession> {
   if (!deprecatedWarningShown && process.env.NODE_ENV !== "production") {
     console.warn(
       "`unstable_getServerSession` has been renamed to `getServerSession`."