Hallvard Breien Furuseth
2015-06-16 10:01:55 UTC
Content preview: Some wishes for RE25: Expose most LDAP_DEPRECATED functions
again. This code is not going away, or it'd be gone by now after 16 years
of deprecation. I usually #define LDAP_DEPRECATED anyway. What's the point
of trying to make me pass 2 extra NULL args to a bunch of functions? And
making a "SASL bind" function the proper way to do Simple Bind is silly and
confusing. OTOH some depreciation make sense, like ldap_get_values() which
returns char** instead of berval**. [...]
Content analysis details: (-1.9 points, 5.0 required)
pts rule name description
---- ---------------------- --------------------------------------------------
-0.0 T_RP_MATCHES_RCVD Envelope sender domain matches handover relay
domain
-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%
[score: 0.0000]
Some wishes for RE25:
Expose most LDAP_DEPRECATED functions again. This code is not going
away, or it'd be gone by now after 16 years of deprecation. I usually
#define LDAP_DEPRECATED anyway. What's the point of trying to make me
pass 2 extra NULL args to a bunch of functions? And making a "SASL
bind" function the proper way to do Simple Bind is silly and confusing.
OTOH some depreciation make sense, like ldap_get_values() which returns
char** instead of berval**.
Add --enable-cldap as "experimental". Add tests and some fixes. The
CLDAP code isn't going away either, so i seems better to support it
properly. People are submitting libldap ITSes, but slapd and ldapsearch
usage remain broken. slapd looks easy to fix, ldapsearch depends on
vestiges of libldap code to ignore Bind requests. Dunno if I want to
bother tracking that down in libldap or just omit the Bind in
ldapsearch. Maybe RedHat has a fix.
...or do the opposite, kill LDAP_DEPRECATED and CLDAP. I hope not, but
that's better than eternally deprecated/code-rotting features.
Kill K&R1 support, i.e. the LDAP_P(()) stuff. Supposedly the headers
support K&R1, but OpenLDAP-2.0 broke this (ITS#6171). Nobody has cared
enough to fix it, and I refuse to fix it.
See if we can require C99 to build OpenLDAP, but keep the headers
C90-compliant. In particular, replace Debug() with a varargs macro,
initially without changing the calls to Debug(). If we get away with
that, we can kill the extra ,0,0,0 arguments to Debug() later so gcc
-Wformat will catch missing/spurious args.
again. This code is not going away, or it'd be gone by now after 16 years
of deprecation. I usually #define LDAP_DEPRECATED anyway. What's the point
of trying to make me pass 2 extra NULL args to a bunch of functions? And
making a "SASL bind" function the proper way to do Simple Bind is silly and
confusing. OTOH some depreciation make sense, like ldap_get_values() which
returns char** instead of berval**. [...]
Content analysis details: (-1.9 points, 5.0 required)
pts rule name description
---- ---------------------- --------------------------------------------------
-0.0 T_RP_MATCHES_RCVD Envelope sender domain matches handover relay
domain
-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%
[score: 0.0000]
Some wishes for RE25:
Expose most LDAP_DEPRECATED functions again. This code is not going
away, or it'd be gone by now after 16 years of deprecation. I usually
#define LDAP_DEPRECATED anyway. What's the point of trying to make me
pass 2 extra NULL args to a bunch of functions? And making a "SASL
bind" function the proper way to do Simple Bind is silly and confusing.
OTOH some depreciation make sense, like ldap_get_values() which returns
char** instead of berval**.
Add --enable-cldap as "experimental". Add tests and some fixes. The
CLDAP code isn't going away either, so i seems better to support it
properly. People are submitting libldap ITSes, but slapd and ldapsearch
usage remain broken. slapd looks easy to fix, ldapsearch depends on
vestiges of libldap code to ignore Bind requests. Dunno if I want to
bother tracking that down in libldap or just omit the Bind in
ldapsearch. Maybe RedHat has a fix.
...or do the opposite, kill LDAP_DEPRECATED and CLDAP. I hope not, but
that's better than eternally deprecated/code-rotting features.
Kill K&R1 support, i.e. the LDAP_P(()) stuff. Supposedly the headers
support K&R1, but OpenLDAP-2.0 broke this (ITS#6171). Nobody has cared
enough to fix it, and I refuse to fix it.
See if we can require C99 to build OpenLDAP, but keep the headers
C90-compliant. In particular, replace Debug() with a varargs macro,
initially without changing the calls to Debug(). If we get away with
that, we can kill the extra ,0,0,0 arguments to Debug() later so gcc
-Wformat will catch missing/spurious args.
--
Hallvard
Hallvard