Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

📚 Documentation: add await to cookies().get() as it returns a promise next.js-ssr-auth demo #1364

Open
2 tasks done
hefler opened this issue Sep 19, 2024 · 0 comments · May be fixed by #1365
Open
2 tasks done

📚 Documentation: add await to cookies().get() as it returns a promise next.js-ssr-auth demo #1364

hefler opened this issue Sep 19, 2024 · 0 comments · May be fixed by #1365
Labels
documentation Improvements or additions to documentation

Comments

@hefler
Copy link

hefler commented Sep 19, 2024

💭 Description

On step 3 of the documentation for Server-side authentication with Next.js on line 11 of the sample code:

const session = cookies().get("my-custom-session");

It might benefit from adding an await on this call, since the method cookies().get() returns a Promise (as per MDN docs).

👀 Have you spent some time to check if this issue has been raised before?

  • I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

@hefler hefler added the documentation Improvements or additions to documentation label Sep 19, 2024
@hefler hefler linked a pull request Sep 19, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant