Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
    Poul Kjeldager Sørensen
    @s093294
    cool. thanks brock
    Poul Kjeldager Sørensen
    @s093294
    Hmm, i dont see any replacement of controller resolver service in webapi, wont it pickup any controller referenced by the app? https://github.com/thinktecture/Thinktecture.IdentityServer.v3/blob/5a6dd546ad437fd1e0213b68f617907f43b3a4c2/source/Core/Configuration/Hosting/WebApiConfig.cs
    Poul Kjeldager Sørensen
    @s093294
    okay. I was under the impression that it was the IHttpControllerTypeResolver service that took care of resolving controllers. Didnt know i could control it on the dependency resolver also
    Brock Allen
    @brockallen
    well, maybe.
    feel free to double-check and then let me know :)
    Poul Kjeldager Sørensen
    @s093294
    Ye, I will see if causes me any trouble and then check it :)
    Poul Kjeldager Sørensen
    @s093294
    @brockallen around? I updated idmgr to beta 3 from nuget, but the Registration Class dont seem complete, dll only has one constructor: public Registration(string name = null);
    is there a newer nuget?
    nvm
    i used the <,> generic version
    form the IDependencyResolver, can i resolve a owin context for the request?
    is it also this resolver that is passed to me in the registration ?
    Poul Kjeldager Sørensen
    @s093294
    Could use OwinEnvironmentService in identity manager also
    Dominick Baier
    @leastprivilege
    not yet
    Brock Allen
    @brockallen
    @s093294 i was just looking at that the other day -- why do you need/want the owin env? it's easy enough to add...
    Poul Kjeldager Sørensen
    @s093294
    @brockallen I want it because we have our own DI on the context that I would like to resolve the usermanager and rolemanager from to pass to idm like i did with idsrv here: https://gist.github.com/s093294/f6c39fcbb1207b760c90
    Brock Allen
    @brockallen
    so we can add the env -- no prob
    i don't understand the comment about the DI being incomplete?
    it's just copied from idsvr
    unless i missed something
    Poul Kjeldager Sørensen
    @s093294
    that was my bad. i used the wronge class
    it would be usefull for me with the environment, but if its not there i will find a different path
    Brock Allen
    @brockallen
    i can add the owin env. i'll go do that now
    Poul Kjeldager Sørensen
    @s093294
    cool.
    Brock Allen
    @brockallen
    done
    Poul Kjeldager Sørensen
    @s093294
    cool. is build being triggered on checkins?
    Brock Allen
    @brockallen
    yes
    on myget
    Poul Kjeldager Sørensen
    @s093294
    super i will use that instead then
    btw i saw you used a PropertyValue, do you know if webapi can deserialize to that from a plain form urlencoded post, or do you fix it on client before posting?
    Brock Allen
    @brockallen
    uhh, wat?
    Poul Kjeldager Sørensen
    @s093294
    nvm :) just me messing around
    Poul Kjeldager Sørensen
    @s093294
    feature request or idea. Would be nice if identity manager did something similar to idsrv when getting claims. Such the phone,email would show as claims. Or alternative on the properties tab, allow the userservice to register additional properties to show in the properties tab
    Brock Allen
    @brockallen
    you can already define properties that draw their value from anywhere
    the property concept is an abstract thing
    and there's a get/set pair for you to do whatever you want
    you just need to do this in the metadata
    making it easier to build custom metadata is a pain now, tho
    so that needs improvement
    Poul Kjeldager Sørensen
    @s093294
    so its possible with current implementation to get the phone and email fields to show on properties tab?
    or would i need to do a PR with it to get it in
    Brock Allen
    @brockallen
    yes -- iirc one of the samples hows how to do that
    it might be in the MR or AspId sample hosts... or even in the core in-mem host
    Poul Kjeldager Sørensen
    @s093294
    i will take a look
    any reason why you left out the email verified and phone verified and username in default implementation
    Poul Kjeldager Sørensen
    @s093294
    ahh includeAccountProperties
    or naa that was deault true