> This patch eliminates the hardcoded charset table. Instead it > reads the charset table from a conf file. The directive > AuthLDAPCharsetConfig allows the admin to specify the charset conf > file. Is there also a need to specify additional conversions > directly in the httpd.conf file through a different directive? It > seems that the charset conf file would be sufficient. If there are > multiple charsets per language, these can be set by specifying the > 5 character language ID rather than the 2 character ID similar to > the example in the charset.conv file for chinese.
As nd said, if someone needs additional conversion, he will scream for it. :-) But something else is going around in my head. Why should this charset conversion be limited to ladp? Well, I don't know where we need the conversion table too. But the table itself should be general available to all modules. Maybe some other modules would like to do the same. A core (?) directive like LanguageCharsetConfig might be much more useful then AuthLDAPCharsetConfig. So the next step would be to move the conversion function to core or apr or so, too. Each module, which needs a conversion, can call this funtion instead of having its own code. Maybe there are some overlapping with mod_charset_lite which also does charset conversion. Kess
