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

nextjs nested middleware is not allowed anymore #5180

Closed
luiisca opened this issue Aug 17, 2022 · 1 comment · Fixed by #5355
Closed

nextjs nested middleware is not allowed anymore #5180

luiisca opened this issue Aug 17, 2022 · 1 comment · Fixed by #5355
Labels
documentation Relates to documentation good first issue Good issue to take for first time contributors

Comments

@luiisca
Copy link

luiisca commented Aug 17, 2022

What is the improvement or update you wish to see?

This section of the next-auth docs contains outdated info about nested middlewares on nextjs

Is there any context that might help us understand?

Tried to follow the advice of putting the _middleware file under a nested directory and runned under this Nested Middleware error

Does the docs page already exist? Please link to it.

https://next-auth.js.org/tutorials/securing-pages-and-api-routes#nextjs-middleware

@luiisca luiisca added documentation Relates to documentation triage Unseen or unconfirmed by a maintainer yet. Provide extra information in the meantime. labels Aug 17, 2022
luiisca added a commit to luiisca/next-auth that referenced this issue Aug 17, 2022
@balazsorban44
Copy link
Member

Good call, care to update in a PR? There's a edit this page button at the end of that page that should take you directly to the file that needs changing. Here is the correct documentation, using matcher instead of nested Middleware https://next-auth.js.org/configuration/nextjs#basic-usage

@balazsorban44 balazsorban44 added good first issue Good issue to take for first time contributors and removed triage Unseen or unconfirmed by a maintainer yet. Provide extra information in the meantime. labels Aug 22, 2022
stphnnnn added a commit to stphnnnn/next-auth that referenced this issue Sep 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Relates to documentation good first issue Good issue to take for first time contributors
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants