Skip to content

Fix Identity cookie redirect paths to match ViewPrefix#81

Merged
antosubash merged 1 commit intomainfrom
claude/fix-docker-auth-1x4E4
Apr 5, 2026
Merged

Fix Identity cookie redirect paths to match ViewPrefix#81
antosubash merged 1 commit intomainfrom
claude/fix-docker-auth-1x4E4

Conversation

@antosubash
Copy link
Copy Markdown
Owner

Summary

  • ASP.NET Identity defaults LoginPath to /Account/Login, but the Users module uses ViewPrefix = "/Identity/Account", placing the login route at /Identity/Account/Login
  • Unauthenticated requests were redirected to /Account/Login which returned 404
  • Explicitly configures LoginPath, LogoutPath, and AccessDeniedPath on the application cookie to match the ViewPrefix

Test plan

  • Deploy and verify unauthenticated API requests redirect to /Identity/Account/Login instead of /Account/Login
  • Verify login page loads and form submission works
  • Verify register page loads and form submission works
  • Verify logout redirects correctly

ASP.NET Identity defaults the login redirect to /Account/Login, but
the Users module's ViewPrefix is /Identity/Account, placing the actual
login route at /Identity/Account/Login. This mismatch caused
unauthenticated requests to redirect to a 404 page.

Explicitly sets LoginPath, LogoutPath, and AccessDeniedPath on the
application cookie to match the module's ViewPrefix.
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying simplemodule-website with  Cloudflare Pages  Cloudflare Pages

Latest commit: a666a43
Status: ✅  Deploy successful!
Preview URL: https://989db037.simplemodule-website.pages.dev
Branch Preview URL: https://claude-fix-docker-auth-1x4e4.simplemodule-website.pages.dev

View logs

@antosubash antosubash merged commit 8e9e0d2 into main Apr 5, 2026
4 checks passed
@antosubash antosubash deleted the claude/fix-docker-auth-1x4E4 branch April 5, 2026 13:46
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