On 5/19/22 5:15 PM, Stefan Eissing wrote:
> 
> 
>> Am 19.05.2022 um 16:44 schrieb Joe Orton <jor...@redhat.com>:
>>
>> On Wed, May 18, 2022 at 05:34:22PM +0200, Ruediger Pluem wrote:
>>> On 5/18/22 4:55 PM, Joe Orton wrote:
>>>> I think for httpd it is only safe and sane to run httpd with LANG=C, we 
>>>> do this in the default service scripts in Fedora/RHEL for a very long 
>>>> time. Other than the protocol parsing issues you can get in non-C 
>>>> locales, you can also get "surprises" when sort order can change with 
>>>> the system locale, impacting e.g. config file load ordering and more.
>>>
>>> Don't you need a locale sensitive case insensitive string comparison in 
>>> case of case blind file systems which support extended
>>> latin characters? I know these Germans with their Umlaute :-).
>>
>> Heh. Well, I got away with it so far :)
>>
>>>> So IMHO it is probably sufficient & simpler to adjust apachectl to set 
>>>> LANG=C rather than trying to eliminate strcasecmp, and add another 
>>>> strcasecmp() reimplementation in APR, in this case.
>>>
>>> We already have this implementation in APR and we use the
>>> httpd one which is just a forward port from APR to httpd until we require a 
>>> sufficient recent APR version in several places.
>>> The question is just if we should use them everywhere and thus do the 
>>> correct thing no matter what locale is set.
>>
>> Ah, I missed that, thanks.
>>
>> +1 from me on doing replacement of strcasecmp() with the 
>> locale-insensitive versions then. At least with config options, protocol 
>> data, it is definitely right.
>>
> 
> +1 from me for replacing our protocol+config handling code with the 
> ap_cstr_casecmp().

+1. Just to mention: Christophe already did quite some work in this area.

Regards

RĂ¼diger

Reply via email to