Re: Exception when no "Conditions" element in assertions

Dave Smith <[email protected]> Fri, 22 Oct 2004 13:58:25 -0600
Newsgroups gmane.comp.sourceid.sso.devel
Message-ID <[email protected]>
Emmanuel,

Per the Liberty 1.2 schema specification, line 500, all IDPs must
include an <AudienceRestrictionCondition/> element. Unfortunately,
this is not enforced by the Liberty schema (or rather, their usage of
SAML schema). :)

I hope this helps,

D.


On Fri, 22 Oct 2004 19:51:28 +0200, Emmanuel Raviart
<[email protected]> wrote:
> We are currently using SourceID Liberty 2.0 beta as a Liberty Alliance
> reference platform to test and debug Lasso, a C library we are developing.
> 
> We are very impressed by the quality of SourceID, but we believe that we
> have found a small bug:
> 
> When using SourceID as a SP and Lasso as and IDP, the authentication
> assertions generated by Lasso don't contain any "Conditions" element.
> 
> If I understand correctly SAML 1.1 documentation, "Conditions" are
> optional in assertions. But SourceID Liberty generates a null pointer
> exception, when it process such an assertion.
> 
> To correct this, we added a test in VerifyAssertions.java, before line 109 :
> ConditionsType conditionsCopy =
> (ConditionsType)assertion.getConditions().copy();
> 
> so that the method "execute" returns immediately when getConditions()
> returns null.
>