* Jack Meyer <
jrm...@gmail.com> [2014-05-08 15:19]:
> I agree it doesn't make any sense. I tried setting the relaystate as
> mentioned in the documentation but that did not work. Though please realize
> that the IDP I am working with is not SimpleSaml, it's ADFS.
SimpleSAMLphp's documetation on IDP-initiated logins doesn't apply
then.
> I tried using a utility from Microsoft "AD FS 2.0 RelayState
> Generator for IDP Initiated Signon" to create the IDP url that
> contains the RelayState and that did not work either.
If you're the SP, why bother about creating RelayState for the IDP?
Whatever the institution does to make people start the session at the
IDP, people will still try to access the SP, and they will bookmark SP
resources. In which case your SP will sent RelayState to the IDP as
needed (when the IDP needs to be involved again) and the IDP is
required by the spec to return the RelayState value verbatim to the
SP.
> Either way, I needed a way to handle an IDP Initiation with no
> RelayState sent
According to the docs for the SP
https://simplesamlphp.org/docs/stable/saml:sp what you did should just
work as expected. I would suggest to file a bug report with all the
necessary details.
> On another note. Many of our clients that want SSO do not allow
> External LDAP connections on port 389, so for SP initiated login for
> LDAP, I had to modify the LDAP ConfigHelper to include a private
> variable $port and in the constructor set it to
> $config->getInteger('port',389), so that I could set different LDAP
> ports for different IDP's via authsources.php.
I have no idea what you're talking about. SAML (and IDP- or
SP-initiated is immaterial here) has nothing to do with LDAP.
And a SAML SP should have no reason to connect to anyone's LDAP except
maybe a data store of its own.
At the risk of stating the obvious: The SAML IDP should provide the
SAML SP with all the requred data in SAML attribute statements.
There simply is no need to access the IDP's LDAP DSA, as SAML has been
designed to render exactly that unnecessary.
-peter