Shibcas and Shibboleth IdP 3.4.6

Jones, Brian brian.jones at ua.edu
Fri Oct 4 10:03:08 EDT 2019


Thanks so much for your thorough response, Scott.  We will dig a little deeper based on the information you have provided.  We'll also post an issue to the Shibcas GitHub for what it's worth.  

If we come up with anything that might be useful to others on this list, we'll post back here.  Thanks again for your work.

Brian


Brian Jones 
Programmer Analyst IV 
Enterprise Development and Application Support, OIT 
The University of Alabama


-----Original Message-----
From: users <users-bounces at shibboleth.net> On Behalf Of Cantor, Scott
Sent: Friday, October 4, 2019 8:30 AM
To: Shib Users <users at shibboleth.net>
Subject: RE: Shibcas and Shibboleth IdP 3.4.6

> This may or may not be an issue anybody here would be able to address, 
> as it appears to be a Shibcas issue, but we wanted to post here just in case.

That is affected, yes. I'll add it to the Release Notes. The need for the security fix necessitated violating the normal rules around API stability for plugins, there just wasn't a good alternative.

It is also a bug in the plugin, because it's using implementation code directly, which is a no-no, and this is why. While the API changes were breaking, it's the use of the ExternalAuthenticationImpl class directly in that flow file that renders it formally incorrect. That isn't specifically why it broke, but it's critical that people stop relying on implementation classes in custom flows.

> Does anyone have any feedback on whether we would still be vulnerable 
> to such an attack?  Or, any suggestions for updating to Shibboleth IdP 
> 3.4.6 while using Shibcas?

The change made should mirror the patch applied to the external-authn-flow.xml file in the redirection step at the top, but it's still an unsupported approach. Direct use of the External flow would be a "supportable" way to do this, alternatively one would need to build one's own copy of the non-API class involved to avoid that dependency.

But the breaking change is to the context class' constructor.

-- Scott

--
For Consortium Member technical support, see https://wiki.shibboleth.net/confluence/x/coFAAg
To unsubscribe from this list send an email to users-unsubscribe at shibboleth.net


More information about the users mailing list