Skip to content
This repository has been archived by the owner on Aug 5, 2022. It is now read-only.

Update opensaml dependency to latest version #27

Closed
dmogn opened this issue Sep 21, 2016 · 7 comments
Closed

Update opensaml dependency to latest version #27

dmogn opened this issue Sep 21, 2016 · 7 comments
Assignees
Milestone

Comments

@dmogn
Copy link
Contributor

dmogn commented Sep 21, 2016

Related with #26.
Maybe latest opensaml version has more parsing capabilities for unusual attributes, like google IDP.

@dmogn dmogn added this to the CE 2.4.5 milestone Sep 21, 2016
@dmogn dmogn self-assigned this Sep 21, 2016
@nynymike
Copy link
Contributor

What makes these attributes unusual?

@dmogn
Copy link
Contributor Author

dmogn commented Sep 21, 2016

@nynymike opensaml cannot parse/extract attr value in this case.
We should switch to latest opensaml version before next steps, I think.

@dmogn
Copy link
Contributor Author

dmogn commented Sep 21, 2016

opensaml does not show errors, but parse empty attr value

@dmogn
Copy link
Contributor Author

dmogn commented Sep 21, 2016

Current dependency version: 2.6.1
Latest V2: 2.6.4
V2 branch support discontinued at july 2016.
The current stable release is 3.2.0

@dmogn
Copy link
Contributor Author

dmogn commented Sep 21, 2016

It will be big change, because of API difference. But opensaml v2 is obsolete and unsupported, so we need do it.

@nynymike
Copy link
Contributor

nynymike commented Sep 22, 2016

Note from Mark Dobrinic:

Only thing I remember is when going from OpenSAML 2.5.x to 2.6.x, 
there was this thing with the MetadataProviders, taking a thread per 
provider. That was quite some thing to integrate.`

@willow9886 willow9886 modified the milestones: CE 3.2.0, CE 2.4.5 Apr 10, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants