IdentityServer3.WsFederation icon indicating copy to clipboard operation
IdentityServer3.WsFederation copied to clipboard

Unable to expose offered claims through metadata

Open bgregorius opened this issue 10 years ago • 7 comments

Looking back in the IdentityServer2 code there used to be a way to expose the offered claims in the wsfed/metadata document. This was done through implementing a custom ClaimsRepository and overriding GetSupportedClaimTypes()

This customization of the metadata document no longer seems possible in IdentityServer3. Are there plans to add this back in? If not I'll have to port the project I am working back to IdentityServer2 and that seems like a step in the wrong direction.

bgregorius avatar Apr 21 '15 19:04 bgregorius

We don't have it right now - and it is not planned so far.

This feature is/was rarely used in v2 or other STS products like ADFS. What's your use case?

leastprivilege avatar Apr 22 '15 07:04 leastprivilege

Our use case in our product is that we pull the metadata then dynamically generate a UI for mapping the claims from the IdP (in this case IdentityServer3) to our system (acting as the SP).

bgregorius avatar Apr 22 '15 13:04 bgregorius

OK - I see the issue. We don't have that right now.

I'll add it to the todo list.

leastprivilege avatar Apr 23 '15 08:04 leastprivilege

Are you still interested in this? If so, you can propose a PR. If not, then we will close this.

brockallen avatar Feb 07 '16 16:02 brockallen

We are still interested in this enhancement.

I have locally hacked some changes in that just add the specific claims in the metadata that I need for my implementation. It would be nice to see these be able to be configured in though.

bgregorius avatar Feb 07 '16 16:02 bgregorius

In all reality, this won't get in unless you can submit a PR. We're swamped with other features/work. Thx.

brockallen avatar Feb 07 '16 16:02 brockallen

@bgregorius If you want to pass across your changes for this, I'd be happy to clean it up and create a pull request.

scottbrady91 avatar May 10 '16 10:05 scottbrady91