Skip to content
This repository has been archived by the owner on Nov 9, 2017. It is now read-only.

Saml2P implementation #139

Closed

Conversation

balajircs
Copy link

Hi,

I have modified the base Thinktecture IdSrv to support SAML2P support. Please merge the changes from change version to base version

@leastprivilege
Copy link
Member

Hi,

thanks a lot.

As I said before - I won't merge that all into the core IdSrv. I rather want a separate assembly and we will create interfaces to that.

But leave it as is for now - I need to have a look at the code first. As soon as I find some time.

@ghost ghost assigned leastprivilege Mar 6, 2013
@balajircs
Copy link
Author

Hi Dominick,

You're welcome. I've hardcoded some values to get identity provider details and default login page of OIOSAML.Net. I've processed the SAML response with XLinq. Please go through the document which I've mentioned about all details.

We need to rewrite some generic code to process the SAML request/response. Hope this will be the initial step to implement SAML 2P in ThinkTecture IdSrv.

Please let me know if you have any clarifications. Thanks for your support and response.

Regards,
Balaji R

Date: Wed, 6 Mar 2013 03:22:35 -0800
From: notifications@github.com
To: Thinktecture.IdentityServer.v2@noreply.github.com
CC: balaji_rcs@hotmail.com
Subject: Re: [Thinktecture.IdentityServer.v2] Saml2P implementation (#139)

Hi,

thanks a lot.

As I said before - I won't merge that all into the core IdSrv. I rather want a separate assembly and we will create interfaces to that.

But leave it as is for now - I need to have a look at the code first. As soon as I find some time.


Reply to this email directly or view it on GitHub.

@ghost ghost assigned brockallen Mar 29, 2013
@brockallen
Copy link
Member

This is now assigned to me. As for looking at it, I have this scheduled for the end of May. Sorry that it's so far away, but the schedule is very busy.

@williamwsmith
Copy link

Sorry to be a pest. I am hoping this is being looked at again. I don't know enough about it to contribute technically.
Is there a new priority to this item?

@leastprivilege
Copy link
Member

Well - the problem is, that the SAML component used in this PR has a license that is incompatible with our license. IOW - SAML support would need to be created as a "plugin" and thus separate from core IdSrv. We don't have resources right now to make that happen.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants