httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Daniel Ruggeri <DRugg...@primary.net>
Subject Re: svn commit: r930125 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy.c modules/proxy/mod_proxy.h modules/proxy/mod_proxy_balancer.c
Date Fri, 16 Jul 2010 00:36:05 GMT
On 7/12/2010 2:17 PM, Jim Jagielski wrote:
> On Jul 11, 2010, at 11:04 AM, Jeff Trawick wrote:
>
>    
>> On Thu, Apr 1, 2010 at 6:48 PM,<niq@apache.org>  wrote:
>>      
>>> Author: niq
>>> Date: Thu Apr  1 22:48:38 2010
>>> New Revision: 930125
>>>
>>> URL: http://svn.apache.org/viewvc?rev=930125&view=rev
>>> Log:
>>> Proxy balancer: support setting error status according to
>>> HTTP response code from a backend.
>>> PR 48939 [Daniel Ruggeri<DRuggeri primary.net>]
>>>        
>>      
>>> +    else if (!strcasecmp(key, "erroronstatus")) {
>>>        
>> Any opinions on "erroronstatus"?  It doesn't feel quite right to me,
>> but I'm not confident about a replacement.
>>
>> Just plain "errorstatus" or "errorstatuscodes"?
>>
>> If normal response handling takes the same action on other status
>> codes (I dunno), maybe "extraerrorstatus" or "extraerrorstatuscodes"
>> would be bettter?
>>
>>      
> proxyseterror?
>
>
>    

If erroronstatus is right out, I like errorstatuscodes best as a 
replacement.

Jeff;
    I will gladly take the tasks you mentioned in STATUS as soon as we 
have consensus on the name of the directive. It may be a lofty goal but 
it would be great if this could make it into 2.2.16.
--
Daniel Ruggeri


Mime
View raw message