[Date Prev][Date Next]
[Chronological]
[Thread]
[Top]
Re: StartTLS URL extension
On Mon, 6 Oct 2008, Michael Ströder wrote:
...
> It's slightly different: From my understanding up to now
> ldap_initialize() itself did not send out a LDAP PDU. So the error
> handling of applications might not be prepared for ldap_initialize()
> causing a real error. This is an incompatible API change.
Ah, I think we're picturing slightly different proposals. I'm picturing
it operating similar to how ldaps:// URIs work currently, where TLS is
negotiated automatically when the connection is actually opened. My
comment about the working being done in ldap_initialize() instead of the
client app was more about the recognizing of the extension in the URI and
marking that URI as needing TLS. I agree that ldap_initialize() should
behave as it currently does, setting up the handle but not opening any
connections.
Hmm, StartTLS implies/requires protocol version 3. It seems unfortunate
that ldap_initialize() wasn't made to default to version 3. Lacking a
change to that, what should happen if you use one of these URIs without
setting the version to 3?
a) error, because you're insane for not setting it already
b) ignore, because things shouldn't just break
c) automatically change version, because confusing people is fun.
Philip Guenther