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

Updated to 2.0, all tests passing #12

Closed
wants to merge 2 commits into from
Closed

Updated to 2.0, all tests passing #12

wants to merge 2 commits into from

Conversation

wtvamp
Copy link

@wtvamp wtvamp commented Dec 3, 2017

Updated to asp.net core 2.0 and identity 2.0 using these guides:

https://docs.microsoft.com/en-us/aspnet/core/migration/1x-to-2x/
https://docs.microsoft.com/en-us/aspnet/core/migration/1x-to-2x/identity-2x

I need some help double checking the cookies configuration - that's something I'm not 100% sure of.

@miltador
Copy link
Owner

miltador commented Dec 4, 2017

Hi @wtvamp,
Thanks for your contribution.

I took a quick look for this.

Tests pass but I encountered a number of issues with trying out the IdentitySample.Mvc project. Register\Login doesn't work, according views need to be updated and probably Configure \ ConfigureServices methods need to be revised.

As for the cookies, there is according section in this article: https://docs.microsoft.com/en-us/aspnet/core/migration/1x-to-2x/identity-2x

I tried that and can't now encounter InvalidOperationException: No IAuthenticationSignInHandler is configured to handle sign in for the scheme: Identity.Application error when trying to login.

Anyway, will dig into it once I have some free time.

@martinjt
Copy link

@wtvamp @miltador has this got anywhere? Tempted to just clone/fix and do my own, but happy to help fix this if there's a commitment to push it out to nuget in a reasonable timeframe

@wtvamp wtvamp closed this by deleting the head repository Oct 9, 2023
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.

3 participants