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

fix: avoid redirect on always public paths #5000

Merged
merged 8 commits into from Jul 23, 2022
Merged

Conversation

balazsorban44
Copy link
Member

Fixes #4848

☕️ Reasoning

What changes are being made? What feature/bug is being fixed here?

🧢 Checklist

  • Documentation
  • Tests
  • Ready to be merged

🎫 Affected issues

Please scout and link issues that might be solved by this PR.

Fixes: INSERT_ISSUE_LINK_HERE

📌 Resources

@vercel
Copy link

vercel bot commented Jul 22, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated
next-auth ⬜️ Ignored (Inspect) Jul 22, 2022 at 6:17PM (UTC)

@github-actions github-actions bot added the core Refers to `@auth/core` label Jul 22, 2022
@balazsorban44 balazsorban44 temporarily deployed to Preview July 22, 2022 17:00 Inactive
@github-actions
Copy link

github-actions bot commented Jul 22, 2022

🎉 Experimental release published 📦️ on npm!

pnpm add next-auth@0.0.0-pr.5000.d742df31
yarn add next-auth@0.0.0-pr.5000.d742df31
npm i next-auth@0.0.0-pr.5000.d742df31

@vercel vercel bot temporarily deployed to Preview July 22, 2022 17:30 Inactive
@balazsorban44 balazsorban44 temporarily deployed to Preview July 22, 2022 17:31 Inactive
@vercel vercel bot temporarily deployed to Preview July 22, 2022 17:38 Inactive
@balazsorban44 balazsorban44 temporarily deployed to Preview July 22, 2022 17:39 Inactive
@balazsorban44 balazsorban44 marked this pull request as ready for review July 22, 2022 18:03
@balazsorban44 balazsorban44 temporarily deployed to Preview July 22, 2022 18:19 Inactive
Copy link
Member

@ThangHuuVu ThangHuuVu left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Refers to `@auth/core`
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Entire site withAuth middleware with custom login routes
2 participants