diff options
author | Jean-Frederic Clere <jfclere@apache.org> | 2022-04-01 12:30:06 +0200 |
---|---|---|
committer | Jean-Frederic Clere <jfclere@apache.org> | 2022-04-01 12:30:06 +0200 |
commit | 05e559da21dac3b5b2d8cb23cd709dafcbc99bcf (patch) | |
tree | d1da3921dbbf363ac2f71e76351285e44225b4ca | |
parent | * In case we see an EOC bucket and there was an error bucket before, use its (diff) | |
download | apache2-05e559da21dac3b5b2d8cb23cd709dafcbc99bcf.tar.xz apache2-05e559da21dac3b5b2d8cb23cd709dafcbc99bcf.zip |
Revert r1899390.
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1899479 13f79535-47bb-0310-9956-ffa450edef68
-rw-r--r-- | CHANGES | 4 | ||||
-rw-r--r-- | modules/proxy/mod_proxy.c | 50 | ||||
-rw-r--r-- | modules/proxy/mod_proxy.h | 2 |
3 files changed, 0 insertions, 56 deletions
@@ -1,10 +1,6 @@ -*- coding: utf-8 -*- Changes with Apache 2.5.1 - *) mod_proxy: Add WorkerBalancerGrowth to allow adding workers to - balancer created dynamically or via "empty" <Proxy balancer://../> - [Jean-Frederic Clere] - *) ab: Add an optional ramp delay when starting concurrent connections so as to not trigger denial of service protection in the network. Report levels of concurrency achieved in cases where the test completes before diff --git a/modules/proxy/mod_proxy.c b/modules/proxy/mod_proxy.c index 47f00e7742..27f3118578 100644 --- a/modules/proxy/mod_proxy.c +++ b/modules/proxy/mod_proxy.c @@ -1633,8 +1633,6 @@ static void * create_proxy_config(apr_pool_t *p, server_rec *s) ps->ppinherit_set = 0; ps->bgrowth = 5; ps->bgrowth_set = 0; - ps->wbgrowth = 0; - ps->wbgrowth_set = 0; ps->req_set = 0; ps->recv_buffer_size = 0; /* this default was left unset for some reason */ ps->recv_buffer_size_set = 0; @@ -1791,8 +1789,6 @@ static void * merge_proxy_config(apr_pool_t *p, void *basev, void *overridesv) ps->req_set = overrides->req_set || base->req_set; ps->bgrowth = (overrides->bgrowth_set == 0) ? base->bgrowth : overrides->bgrowth; ps->bgrowth_set = overrides->bgrowth_set || base->bgrowth_set; - ps->wbgrowth = (overrides->wbgrowth_set == 0) ? base->wbgrowth : overrides->wbgrowth; - ps->wbgrowth_set = overrides->wbgrowth_set || base->wbgrowth_set; ps->max_balancers = overrides->max_balancers || base->max_balancers; ps->bal_persist = overrides->bal_persist; ps->recv_buffer_size = (overrides->recv_buffer_size_set == 0) ? base->recv_buffer_size : overrides->recv_buffer_size; @@ -2668,21 +2664,6 @@ static const char *set_bgrowth(cmd_parms *parms, void *dummy, const char *arg) return NULL; } -static const char *set_wbgrowth(cmd_parms *parms, void *dummy, const char *arg) -{ - proxy_server_conf *psf = - ap_get_module_config(parms->server->module_config, &proxy_module); - - int growth = atoi(arg); - if (growth < 0 || growth > 1000) { - return "WorkerBalancerGrowth must be between 0 and 1000"; - } - psf->wbgrowth = growth; - psf->wbgrowth_set = 1; - - return NULL; -} - static const char *set_persist(cmd_parms *parms, void *dummy, int flag) { proxy_server_conf *psf = @@ -3066,35 +3047,6 @@ static const char *proxysection(cmd_parms *cmd, void *mconfig, const char *arg) return apr_pstrcat(cmd->temp_pool, thiscmd->name, " ", err, " ", word, "=", val, "; ", conf->p, NULL); } - } else { - /* we have an empty <Proxy/> */ - if (!ap_strchr_c(conf->p, ':')) - return apr_pstrcat(cmd->pool, thiscmd->name, - "> arguments are not supported for non url.", - NULL); - if (ap_proxy_valid_balancer_name((char *)conf->p, 9)) { - balancer = ap_proxy_get_balancer(cmd->pool, sconf, conf->p, 0); - if (!balancer) { - err = ap_proxy_define_balancer(cmd->pool, &balancer, - sconf, conf->p, "/", 0); - if (err) - return apr_pstrcat(cmd->temp_pool, thiscmd->name, - " ", err, NULL); - } - if (!balancer->section_config) { - balancer->section_config = new_dir_conf; - } - /* Allow to add members dynamically */ - if (sconf->wbgrowth_set) { - balancer->growth = sconf->wbgrowth; - balancer->growth_set = 1; - } else { - return apr_pstrcat(cmd->temp_pool, thiscmd->name, "/> requires WorkerBalancerGrowth > 0.", NULL); - } - } else { - return apr_pstrcat(cmd->pool, thiscmd->name, " " ,conf->p, - "> only balancer://balancername is supported.", NULL); - } } cmd->path = old_path; @@ -3157,8 +3109,6 @@ static const command_rec proxy_cmds[] = "A balancer name and scheme with list of params"), AP_INIT_TAKE1("BalancerGrowth", set_bgrowth, NULL, RSRC_CONF, "Number of additional Balancers that can be added post-config"), - AP_INIT_TAKE1("WorkerBalancerGrowth", set_wbgrowth, NULL, RSRC_CONF, - "Number of additional Workers per Balancer that can be added post-config"), AP_INIT_FLAG("BalancerPersist", set_persist, NULL, RSRC_CONF, "on if the balancer should persist changes on reboot/restart made via the Balancer Manager"), AP_INIT_FLAG("BalancerInherit", set_inherit, NULL, RSRC_CONF, diff --git a/modules/proxy/mod_proxy.h b/modules/proxy/mod_proxy.h index 8281edd14d..2eac740dac 100644 --- a/modules/proxy/mod_proxy.h +++ b/modules/proxy/mod_proxy.h @@ -163,7 +163,6 @@ typedef struct { int req; /* true if proxy requests are enabled */ int max_balancers; /* maximum number of allowed balancers */ int bgrowth; /* number of post-config balancers can added */ - int wbgrowth; /* number of post-config work per balancer that can added */ enum { via_off, via_on, @@ -199,7 +198,6 @@ typedef struct { unsigned int proxy_status_set:1; unsigned int source_address_set:1; unsigned int bgrowth_set:1; - unsigned int wbgrowth_set:1; unsigned int bal_persist:1; unsigned int inherit:1; unsigned int inherit_set:1; |