Possible issue with 2FA

[odinnix]

Hey;

We are seeing an issue with 2FA and I'm not sure if its an Identity change or an issue with this provider.

aspnet/Identity#1383 (comment)

Any thoughts?

[mrahhal]

Hey,

I've not given 2FA too much attention, so it's possible. But because I mostly don't use it, it'd be great if you can provide a small repro.

But the problem itself looks simple, so in any case, I'll check if I can figure something out.

Did it work before v2.0 of Identity? And make sure you're using latest version of this lib.

[odinnix]

Everything worked before Identity 2.0 yep, and am using 3.0.1

I didn't actually setup the 2FA piece but I will chat with the developer who did to see if we can get a sample repo.

Thanks!

[mrahhal]

I see. For now, a stacktrace will be most helpful.

[mrahhal]

So upon first inspection, I guess IUserAuthenticatorKeyStore is a new interface in 2.0. Of course, I don't implement this anywhere so I'm guessing I'll have to implement it on UserStore. There might be other new interfaces too, so I'll take this chance to check up on the EFCore impl and match them up.

[odinnix]

Awesome, thank you for looking into this.

If you need me to test I can pull a prerelease package and try it out.

Do you still need a stack trace? Can get you one in a few hours here once the dev is available.

[mrahhal]

Probably won't need one, as it's obvious I simply don't implement the mentioned interface.

I'll ask you to test when I finish something. Thanks.

[mrahhal]

It appears that the architectural changes in Identity 2.0 are actually big. I made a hasty decision when I released 3.0. Looks like now there are also base stores that can be used, instead of having them all in the same store project.

For later, I'm thinking about making a new repo with a matching version and better package name: "MR.AspNetCore.Identity.EntityFramework6", that will use the stores provided by Identity Core. And this one kept for aspnetcore Identity 1.0.

For now, I'll release a quick fix.

@dcarl1 can you try this one out and let me know if it works for you?

[akonyer]

Tried your pre-release and now I'm getting a different error:

error: The entity type IdentityUserToken is not part of the model for the current context.

[odinnix]

@mrahhal @akonyer is the same project :)

[mrahhal]

Yes wanted to warn about it but forgot. You'll need to do an EF migration. Since there's a new entity (IdentityUserToken) used for 2FA and other stuff.

[mrahhal]

Ah wait. Did I not add it to the context. 🤔 I'll check.

[mrahhal]

Sure enough, I forgot to add it to the context (I don't have a handy test host for 2FA so really sorry).
I'll have to ask you to test again. Here is the new release.

[akonyer]

Well, we're making progress I think?

{"error":"One or more validation errors were detected during model generation:

EFStorageProvider.IdentityUserToken: : EntityType 'IdentityUserToken' has no key defined. Define the key for this EntityType.
UserTokens: EntityType: EntitySet 'UserTokens' is based on type 'IdentityUserToken' that has no keys defined.
"}

[mrahhal]

Uh.. It seems an IdentityUserToken has a composite key.

[mrahhal]

Here you go.