Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use OpenSAML V3 as base library #7

Open
kaunio opened this issue Feb 8, 2016 · 4 comments
Open

Use OpenSAML V3 as base library #7

kaunio opened this issue Feb 8, 2016 · 4 comments

Comments

@kaunio
Copy link
Contributor

kaunio commented Feb 8, 2016

Support for OpenSAML V2 will cease to exist on July 31, 2016 (see https://wiki.shibboleth.net/confluence/display/OpenSAML/Home).

It would be nice if this library was adjusted to support the latest version so it can be used for a while in the feature and still rely on supported libraries.

@kaunio
Copy link
Contributor Author

kaunio commented Feb 8, 2016

I've started work with an SP implementation that is using this library so I will rewrite it tomorrow anyway. Just wanted to file a ticket to see if there are any interest from others with the OpenSAML V3 support.

V2 and V3 is quite different initialization wise (a lot of changes in how they work with factories and such) so it's not going to be easy to support a code base that can work with both versions at the same time.

@bcopeland
Copy link
Contributor

On Mon, Feb 08, 2016 at 09:55:54AM -0800, Jerker Klang wrote:

Support for OpenSAML V2 will cease to exist on July 31, 2016 (see https://wikishibbolethnet/confluence/display/OpenSAML/Home)

It would be nice if this library was adjusted to support the latest version so it can be used for a while in the feature and still rely on supported libraries

Yes, I think we'll want to switch to the V3 library before V2 is EOL.

Bob Copeland %% http://bobcopeland.com/

@kaunio
Copy link
Contributor Author

kaunio commented Feb 9, 2016

I've now done the bulk of the work in this branch https://github.com/kaunio/saml-sdk-java/tree/feature-opensamlv3 .

I've not created a pull request yet since I'm waiting for #9 to be merged to master.

I just want to rise some concerns that the upgrade might create some compatibility issues with other users. However it's most likely not an issue because I think that OpenSAML V2 and V3 can coexist. They seems to use separate packages.

@ekarlso
Copy link

ekarlso commented Aug 17, 2016

Are there any ideas on updating this to v3 ?

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

No branches or pull requests

3 participants