httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jan Kaluža <jkal...@redhat.com>
Subject Re: [PATCH] Fix settings options with ProxyPassMatch
Date Wed, 19 Mar 2014 09:24:25 GMT
On 03/19/2014 10:13 AM, Jan Kaluža wrote:
> On 03/19/2014 09:59 AM, Jan Kaluža wrote:
>> On 03/18/2014 02:46 PM, Yann Ylavic wrote:
>>> On Tue, Mar 18, 2014 at 2:38 PM, Yann Ylavic <ylavic.dev@gmail.com>
>>> wrote:
>>>> Wouldn't it be possible to define wildcard workers when the URL is
>>>> known to be a regexp substitution?
>>>> For these workers' URLs, the dollars (plus the following digit) could
>>>> be replaced by a wildcard (ie. *) and ap_proxy_get_worker() could then
>>>> use ap_strcasecmp_match() against the requested URL.
>>>
>>> I meant ap_strcmp_match(), this has to be case sensitive...
>>>
>>
>> I've implemented your idea. Can you check the attached patch please? It
>> fixes the original PR and also ProxyPassMatch with UDS for me.
>
> Ignore this patch. It breaks ProxyPass. Attached is better version.

Not my day today. s/apr_isxdigit/apr_isdigit.

>
>> If it's OK, I will commit it.
>>
>> Regards,
>> Jan Kaluza
>>
>
> Regards,
> Jan Kaluza
>


Mime
View raw message