AuthenticateLocalAsync is not virtual anymore

IdentityServer / IdentityServer3.AspNetIdentity

ASP.NET Identity support for Thinktecture IdentityServer3

Apache License 2.0

64 stars 51 forks source link

AuthenticateLocalAsync is not virtual anymore #23

Closed soliveira closed 9 years ago

soliveira commented 9 years ago

Hi.

I've Updated Thinktecture.IdentityServer.v3.AspNetIdentity to beta4 and noticed that the method AuthenticateLocalAsync is no longer virtual is there any reason why?

brockallen commented 9 years ago

Nope, no reason -- probably an oversight. I'll fix it. thanks

brockallen commented 9 years ago

I just looked... I don't think it was ever virtual. But anyway, I'll mark them as such.

brockallen commented 9 years ago

done on dev branch. you can reference the assembly on our myget feed

soliveira commented 9 years ago

Thanks, It was virtual: https://github.com/thinktecture/Thinktecture.IdentityServer.v3.AspNetIdentity/commit/66559789c99b9084816b6492d1ee3839db7e4e80#diff-9e3e6ec7773f20938c9fe06590b11ea5