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

Update the 2.1 authentication examples -> at Startup.cs .AddDefaultUI() method is missing #6967

Closed
hubekpeter opened this issue Jun 9, 2018 · 5 comments
Labels
Pri1 High priority, do before Pri2 and Pri3 Pri2 Priority 2 Source - Docs.ms Docs Customer feedback via GitHub Issue
Milestone

Comments

@hubekpeter
Copy link

Since 2.1 the identity views have been moved under the separate area. The examples would not work without .AddDefaultUI() method been called.

services.AddIdentity<IdentityUser, IdentityRole>()
.AddEntityFrameworkStores()
.AddDefaultTokenProviders()
.AddDefaultUI();

Thanks
P.

@guardrex
Copy link
Collaborator

guardrex commented Jun 9, 2018

Which topic @hubekpeter?

@guardrex guardrex added the Source - Docs.ms Docs Customer feedback via GitHub Issue label Jun 9, 2018
@hubekpeter
Copy link
Author

hubekpeter commented Jun 9, 2018 via email

@guardrex
Copy link
Collaborator

guardrex commented Jun 9, 2018

I was just taking a look at the issues, and there are several that pertain to updating Identity for 2.1. Those issues should cover this one. The work is going on now, and I doubt it will take long to get the topics updated.

https://github.com/aspnet/Docs/issues?utf8=%E2%9C%93&q=is%3Aissue+is%3Aopen+identity

Sample updates are tracked by #5495.

@guardrex guardrex added this to the Backlog milestone Jun 9, 2018
@samisin
Copy link

samisin commented Jul 6, 2018

Did run into the same issue when using a customized Identity. If services.AddDefaultIdentity is used there is no issue with the separate area, but as soon as services.AddIdentity is used, AddDefaultUI needs to be added to get correct routing. Otherwise (it not authorized, for example) it routs to /Account/Login and not to /Identity/Account/Login. My two cents.

@Rick-Anderson Rick-Anderson added Pri1 High priority, do before Pri2 and Pri3 Pri2 Priority 2 labels Oct 26, 2018
@Rick-Anderson
Copy link
Contributor

Dup of #9706, #9706, #10418

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Pri1 High priority, do before Pri2 and Pri3 Pri2 Priority 2 Source - Docs.ms Docs Customer feedback via GitHub Issue
Projects
None yet
Development

No branches or pull requests

4 participants