Bug 43167 - Adding status=i|I (Ignore Errors) option for ProxyPass
Summary: Adding status=i|I (Ignore Errors) option for ProxyPass
Status: RESOLVED FIXED
Alias: None
Product: Apache httpd-2
Classification: Unclassified
Component: mod_proxy (show other bugs)
Version: 2.2.4
Hardware: All All
: P2 enhancement (vote)
Target Milestone: ---
Assignee: Apache HTTPD Bugs Mailing List
URL:
Keywords: PatchAvailable
Depends on:
Blocks:
 
Reported: 2007-08-20 06:29 UTC by Francisco Gimeno
Modified: 2007-09-08 06:10 UTC (History)
0 users



Attachments
proxypass_status_worker_ignore_errors_options.diff (672 bytes, patch)
2007-08-20 06:33 UTC, Francisco Gimeno
Details | Diff
Add the 'I' option to ProxyPass status directive (872 bytes, patch)
2007-08-30 13:10 UTC, Francisco Gimeno
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Francisco Gimeno 2007-08-20 06:29:28 UTC
Hello,

I have made a tiny patch for allowing marking a ProxyPass entry as
WORKER_IGNORE_ERRORS. That will allow not to disable the requests to the server
if it makes a Timeout or  single error.

---
diff -ur apache2-2.2.4.orig/modules/proxy/mod_proxy.c
apache2-2.2.4.kikov1/modules/proxy/mod_proxy.c
--- apache2-2.2.4.orig/modules/proxy/mod_proxy.c        2006-11-27
16:00:56.000000000 +0100
+++ apache2-2.2.4.kikov1/modules/proxy/mod_proxy.c      2007-08-20
15:17:46.000000000 +0200
@@ -219,6 +219,12 @@
                 else
                     worker->status &= ~PROXY_WORKER_HOT_STANDBY;
             }
+           else if (*v == 'I' || *v == 'i') {
+               if (mode)
+                   worker->status |= PROXY_WORKER_IGNORE_ERRORS;
+               else
+                   worker->status &= ~PROXY_WORKER_IGNORE_ERRORS;
+           }
             else {
                 return "Unknown status parameter option";
             }
---

I have found people looking for this:
 http://www.mail-archive.com/dev@httpd.apache.org/msg36492.html
Comment 1 Francisco Gimeno 2007-08-20 06:33:18 UTC
Created attachment 20681 [details]
proxypass_status_worker_ignore_errors_options.diff

Patch on 2.2.4.. It should apply perfectly in others (tiny and clean diff)
Comment 2 William A. Rowe Jr. 2007-08-21 02:55:00 UTC
Not reviewed, just tagging.
Comment 3 Francisco Gimeno 2007-08-30 13:10:43 UTC
Created attachment 20743 [details]
Add the 'I' option to ProxyPass status directive

Update on documentation. That patch has been done with svn diff to trunk
Comment 4 Nick Kew 2007-09-08 06:10:08 UTC
Fixed in 2.2.6