search

IdentityServer / IdentityServer3.AspNetIdentity

ASP.NET Identity support for Thinktecture IdentityServer3
Apache License 2.0
64 stars 51 forks source link

What version of IdentityServer is being used in this sample? #13

Closed donmiller714 closed 10 years ago

donmiller714 commented 10 years ago

I've tried to modify the source from Thinktecture.IdentityServer.v3 and find the implementation has created some breaking changes to this sample:

-> Scope.Type and the ScopeTypes constants replaced with IsOpenIdScope. -> AuthenticateExternalAsync has been added and must be implemented in AspNetIdentityUserService.

Not a big deal, I modified the code accordingly and it compiles. To build Thinktecture.IdentityServer.dll I copied part of the the default.ps1 script to do just the ilmerge bit and create the merged dll, but now when I run the sample I'm getting:

"Could not load type 'System.Net.Http.HttpMessageHandler' from assembly 'Thinktecture.IdentityServer, Version=1.0.0.0, Culture=neutral, PublicKeyToken=null' because the method '.ctor' has no implementation (no RVA)."

The service still runs but when you try to call it it breaks in Owin.UseIdentityServerExtension. I poked around a bit to see if I can find the version of IdentityServer that was used for the sample but I can't. Can someone point me in the right direction?

thanks!

brockallen commented 10 years ago

What branch are you using?

donmiller714 commented 10 years ago

The master, assuming that's the branch that displays by default.

brockallen commented 10 years ago

We try to have the master stable with the latest release on NuGet. We just updated to beta2, so perhaps the latest on master will be more up to date for you. Mind giving it a shot for rme?

donmiller714 commented 10 years ago

Sure, I'll check it out in the morning. Thanks for the update!

Sent from my iPhone

On Oct 9, 2014, at 1:51 PM, Brock Allen notifications@github.com wrote:

We try to have the master stable with the latest release on NuGet. We just updated to beta2, so perhaps the latest on master will be more up to date for you. Mind giving it a shot for rme?

— Reply to this email directly or view it on GitHub.

brockallen commented 10 years ago

Any update?

donmiller714 commented 10 years ago

No, I'm sorry! However it remains a top priority. Thanks!

On Tue, Oct 14, 2014 at 1:01 PM, Brock Allen notifications@github.com wrote:

Any update?

— Reply to this email directly or view it on GitHub https://github.com/thinktecture/Thinktecture.IdentityServer.v3.AspNetIdentity/issues/13#issuecomment-59106907 .

donmiller714 commented 10 years ago

Hi Brock,

I downloaded the latest versions and preliminary results are that the versions match. I'm still trying to figure out how to get claims back from the ASP .Net Identity using the manual code client but haven't been able to do it yet. Once I figure that out (or you could just tell me!) then I'd say we're good to go.

thanks! Don

On Tue, Oct 14, 2014 at 1:01 PM, Brock Allen notifications@github.com wrote:

Any update?

— Reply to this email directly or view it on GitHub https://github.com/thinktecture/Thinktecture.IdentityServer.v3.AspNetIdentity/issues/13#issuecomment-59106907 .

brockallen commented 10 years ago

Ok, thx for the update.