Comment 4 for bug 614004

On 8/7/10 10:59, Lorenzo Gil Sanchez wrote:
> You introduced a small subtle bug at http://bazaar.launchpad.net
> /~roland-hedberg/pysaml2/main/revision/429 when creating the logout
> request:
>
> request = samlp.LogoutRequest(
> id=sid(),
> version=VERSION,
> issue_instant=instant(),
> destination=destination,
> issuer=self.issuer(),
> session_index=sid(),
> name_id = name_id,
> )
>
> This gives me the following exception when serializing this object into
> a string:
>
True, if anything session_index is an instance of samlp.SessionIndex.
Actually if my reading is correct it should be the session index that
the IdP uses for the login.
If you leave it out, I guess the IdP just closes the login session/-s
that this subject has due to a request from this SP.

-- Roland