Re: mod_jk / mod_jk2 : help from specialists welcome

2004-03-03 Thread jean-frederic clere
Henri Gomez wrote: Hi to all, I'm involved in jk/jk2 on tomcat and we wonder on tomcat-dev if we should use translate in MIDDLE or FIRST position (specifying that mod_rewrite to be the first in hooks chain). I see in jk that we're using :

Re: mod_jk / mod_jk2 : help from specialists welcome

2004-03-03 Thread jean-frederic clere
Justin Erenkrantz wrote: --On Wednesday, March 3, 2004 10:16 AM +0100 jean-frederic clere [EMAIL PROTECTED] wrote: MIDDLE helps to fix 21546... But I do not see why ;-( Oh, *now* I understand the context to Henri's question. Here's a suggestion. And, no, I don't think changing

Re: Has mod_dir changed between 2.0.46 and 2.0.49 ?

2004-04-20 Thread jean-frederic clere
Peter Van Biesen wrote: Hi, I've recently migrated from 2.0.46 to 2.0.49 and I have a strange problem. In 2.0.46, my DirectoryIndex was set to include index.jsp, so if a directory contains a index.jsp, tomcat was contacted to serve this. However, when I do this in 2.0.49 tomcat is contacted even

libtool (jlibtool)

2004-07-06 Thread jean-frederic clere
Hi, I am using jlibtool and I have found something weird in httpd: In Makefile.in: +++ cp build/*.mk $(DESTDIR)$(installbuilddir); \ sed 's#LIBTOOL = \(.*\)#LIBTOOL = $(SHELL) $(installbuilddir)/libtool $(LTFLAGS)#' \ build/config_vars.mk

Re: libtool (jlibtool)

2004-07-07 Thread jean-frederic clere
Joe Orton wrote: On Tue, Jul 06, 2004 at 03:35:30PM +0200, jean-frederic clere wrote: I am using jlibtool and I have found something weird in httpd: In Makefile.in: +++ cp build/*.mk $(DESTDIR)$(installbuilddir); \ sed 's#LIBTOOL = \(.*\)#LIBTOOL = $(SHELL) $(installbuilddir

[PATCH] mod_charset_lite.c

2004-07-07 Thread jean-frederic clere
Hi, When using mod_jk2 and mod_charset_lite mod_charset_lite cores because r-filename is NULL. The easy patch is attached. Cheers Jean-Frederic Index: modules/experimental/mod_charset_lite.c === RCS file:

Re: libtool (jlibtool)

2004-07-09 Thread jean-frederic clere
Joe Orton wrote: On Wed, Jul 07, 2004 at 09:44:00AM +0200, jean-frederic clere wrote: Joe Orton wrote: On Tue, Jul 06, 2004 at 03:35:30PM +0200, jean-frederic clere wrote: The aim is to ensure that the LIBTOOL variable in the installed config_vars.mk points at the installed copy of the libtool

Re: [PATCH] mod_charset_lite.c

2004-07-09 Thread jean-frederic clere
Jeff Trawick wrote: jean-frederic clere wrote: Hi, When using mod_jk2 and mod_charset_lite mod_charset_lite cores because r-filename is NULL. patch looks reasonable... unfortunately, I lifted some bad style from somewhere (the code around your patch)... I'll fix that up and then commit

Re: Invitation to HTTPD commiters in tomcat-dev

2004-07-21 Thread jean-frederic clere
Graham Leggett wrote: Mladen Turk wrote: I don't think that it is necessary for a mod_ajp to be included inside the mod_proxy, although they are sharing some common concepts. I think it's very necessary - sharing those common concepts ultimately makes for doing things in a consistent way. It

Re: [AJP] proxy status

2004-08-13 Thread jean-frederic clere
Guenter Knauf wrote: Hi, That seems rational to me. The reason for proposing [EMAIL PROTECTED] is so that tomcat-dev'ers wouldn't have to swallow the full bandwidth of [EMAIL PROTECTED] (converse of the problem where they asked anyone in [EMAIL PROTECTED] to follow [EMAIL PROTECTED] for the

[PATCH] proxy_ajp.c

2004-08-13 Thread jean-frederic clere
Hi, I have arranged the logic to send the request and the body to Tomcat, now it works for both chunked and not-chunked. What is not yet working is when the Tomcat starts to send data before having all the body and then reads a little more body data. Cheers Jean-Frederic Index: proxy_ajp.c

Re: proxy CONNECT and recent 2.1-dev proxy enhancements

2004-08-16 Thread jean-frederic clere
Jeff Trawick wrote: probably obvious to those doing the load balancing work, but just in case... url in this case is ip:port, which ap_proxy_get_balancer() doesn't handle simple working testcase in case this helps testing connect and send CONNECT 0.0.0.0:8080\r\n Do we need balancer for CONNECT?

Re: proxy CONNECT and recent 2.1-dev proxy enhancements

2004-08-16 Thread jean-frederic clere
Jeff Trawick wrote: On Mon, 16 Aug 2004 12:16:02 +0200, jean-frederic clere [EMAIL PROTECTED] wrote: Jeff Trawick wrote: probably obvious to those doing the load balancing work, but just in case... url in this case is ip:port, which ap_proxy_get_balancer() doesn't handle simple working testcase

Re: [proxy] New implementation ready for testing

2004-08-16 Thread jean-frederic clere
Pier Fumagalli wrote: On 11 Aug 2004, at 17:14, Mladen Turk wrote: Hi all, We've finished the initial development of extended mod_proxy. Since the development took place at jakata-tomcat-connectors, the source code can be found under ajp/proxy. Here is the list of major features added: 3. Added

[PATCH] proxy_connect.c

2004-08-16 Thread jean-frederic clere
Hi, While arranging the code of the recent 2.1-dev proxy enhancements to get CONNECT working I have noted that CONNECT loops endless when the client closes the connection. Find enclosed a patch than fixes the problem. Cheers Jean-Frederic Index: proxy_connect.c

Re: proxy CONNECT and recent 2.1-dev proxy enhancements

2004-08-16 Thread jean-frederic clere
Jeff Trawick wrote: On Mon, 16 Aug 2004 13:20:41 +0200, jean-frederic clere [EMAIL PROTECTED] wrote: Jeff Trawick wrote: On Mon, 16 Aug 2004 12:16:02 +0200, jean-frederic clere [EMAIL PROTECTED] wrote: Jeff Trawick wrote: probably obvious to those doing the load balancing work, but just in case

Re: Where is the AJP code...

2004-08-17 Thread jean-frederic clere
William A. Rowe, Jr. wrote: At 03:58 PM 8/16/2004, Brad Nicholes wrote: The addition of the proxy_ajp module to the httpd project presents a build problem. The AJP module code exists in httpd/modules/proxy in the httpd project but the AJP lib code still sits in the

Re: proxy compile warnings

2004-08-18 Thread jean-frederic clere
Geoffrey Young wrote: William A. Rowe, Jr. wrote: At 04:36 PM 8/17/2004, Joe Orton wrote: On Tue, Aug 17, 2004 at 11:48:39AM -0400, Geoffrey Young wrote: hi all the attached patch is required for me to get mod_proxy to compile with -Werror. Thanks Geoff. The only thing I'm not sure of is

Re: Where is the AJP code...

2004-08-18 Thread jean-frederic clere
William A. Rowe, Jr. wrote: At 01:56 AM 8/17/2004, jean-frederic clere wrote: William A. Rowe, Jr. wrote: At 03:58 PM 8/16/2004, Brad Nicholes wrote: The addition of the proxy_ajp module to the httpd project presents a build problem. The AJP module code exists in httpd/modules/proxy in the httpd

AJP lib files added to httpd-2.0 repos

2004-08-18 Thread jean-frederic clere
Hi, I have added to the httpd-2.0/modules/proxy the following files: ajp.h ajp_header.c ajp_header.h ajp_link.c ajp_msg.c They are also in jakarta-tomcat-connectors/ajp/ajplib/test. They have to be kept up to date (synchronized) please remember to mail the diff to the other repos dev-list when

vacation

2004-08-18 Thread jean-frederic clere
Hi, I'll be on vacation til 7th september. I will (try to) go on the new proxy implementation but I will only read my apache.org mail, so if you find something wrong in the new code (and want a quick fix) mail me there. Cheers Jean-Frederic

Re: [PATCH] don't crash with per-dir (location) rewrite config and NULL r-filename

2004-09-10 Thread jean-frederic clere
Jeff Trawick wrote: See attached patch. Given a module with map-to-storage hook which leaves r-filename NULL, and config like the following, you get segfault on platforms that don't like strlen(NULL). Location /silly/foo RewriteEngine On RewriteCond %{SERVER_PORT} ^8080$ RewriteRule (.*)

Re: cvs commit: httpd-2.0/os/unix os.h unixd.c

2004-10-04 Thread jean-frederic clere
Jeff Trawick wrote: On 1 Oct 2004 16:03:09 -, [EMAIL PROTECTED] [EMAIL PROTECTED] wrote: jfclere 2004/10/01 09:03:09 Modified:os config.m4 os/unix os.h unixd.c Log: Move the few BS2000 specific in unixd.c Index: unixd.c

PATCH to use apr-iconv

2004-10-04 Thread jean-frederic clere
Hi, I have prepared a patch to use apr-iconv instead GNU or system iconv. Find it enclosed. Any comments? Cheers Jean-Frederic Index: buildconf === RCS file: /home/cvspublic/httpd-2.0/buildconf,v retrieving revision 1.44 diff -u

Re: PATCH to use apr-iconv

2004-10-05 Thread jean-frederic clere
William A. Rowe, Jr. wrote: At 06:38 PM 10/4/2004, Justin Erenkrantz wrote: I have some issues with the proposed patch in that it moves some configure logic that really belongs in apr-util over to httpd: i.e. configuration of apr-iconv should be done by apr-util not by httpd, httpd should only be

Re: mod_proxy_ajp and ssl

2006-10-19 Thread Jean-frederic Clere
fredk2 wrote: Hi: I have a rhetorical question for the developers of mod_proxy_ajp (and mod_jk). Assuming the tomcat ajp connector was able to accept ssl connections - if the apache httpd server and tomcat are on separate machines and you needed to secure the connection with ssl, would you

module that doesn't want to be shared.

2006-10-20 Thread Jean-frederic Clere
Hi, I have added a modules to httpd-trunk but I have not able to get it shared. (modules.mk shared = is empty!). Has anyone had that problem? Are there restrictions in the name of modules? my config5.m4 contains: +++ APACHE_MODPATH_INIT(proxy_cluster) proxy_cluster_objs=mod_proxy_cluster.lo

Re: module that doesn't want to be shared.

2006-10-20 Thread Jean-frederic Clere
On Fri, 2006-10-20 at 11:53 +0200, Jean-frederic Clere wrote: Hi, I have added a modules to httpd-trunk but I have not able to get it shared. (modules.mk shared = is empty!). I have removed my copy of the repos, checkout again copy back my module files. And now it works. Cheers Jean

Re: module that doesn't want to be shared.

2006-10-20 Thread Jean-frederic Clere
On Fri, 2006-10-20 at 07:30 -0400, Eric Covener wrote: On 10/20/06, Jean-frederic Clere [EMAIL PROTECTED] wrote: On Fri, 2006-10-20 at 11:53 +0200, Jean-frederic Clere wrote: Hi, I have added a modules to httpd-trunk but I have not able to get it shared. (modules.mk shared

Adding a variable size for AJP messages

2006-10-20 Thread Jean-frederic Clere
Hi, I am adding the feature of mod_jk max_packet_size to httpd-proxy-ajp. (in trunk for the moment). Any comments? Cheers Jean-Frederic

Re: Adding a variable size for AJP messages

2006-10-20 Thread Jean-frederic Clere
On Fri, 2006-10-20 at 14:54 +0200, Jean-frederic Clere wrote: Hi, I am adding the feature of mod_jk max_packet_size to httpd-proxy-ajp. (in trunk for the moment). Any comments? To set the max_packet_size ProxyIOBufferSize seems to the best parameter to use. Any comments? Cheers Jean

Re: svn commit: r467259 - /httpd/httpd/trunk/modules/proxy/ajp_utils.c

2006-10-24 Thread Jean-frederic Clere
On Tue, 2006-10-24 at 07:01 +, [EMAIL PROTECTED] wrote: Author: mturk Date: Tue Oct 24 00:01:44 2006 New Revision: 467259 URL: http://svn.apache.org/viewvc?view=revrev=467259 Log: 128 is enough for CPING/CPONG messages. Was 3 too small? Cheers Jean-Frederic Modified:

Re: svn commit: r467014 - in /httpd/httpd/trunk/modules/proxy: ajp.h

2006-10-24 Thread Jean-frederic Clere
On Mon, 2006-10-23 at 14:45 -0400, Jim Jagielski wrote: [EMAIL PROTECTED] wrote: Modified: httpd/httpd/trunk/modules/proxy/ajp.h URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/proxy/ajp.h?view=diffrev=467014r1=467013r2=467014

Adding a variable size for AJP messages

2006-10-25 Thread Jean-frederic Clere
Hi, I am adding the feature of mod_jk max_packet_size to httpd-proxy-ajp. (in trunk for the moment). Any comments? Cheers Jean-Frederic

httpd-proxy-scoreboard how to go on

2006-12-15 Thread Jean-Frederic Clere
Hi, I would like to return the httpd-proxy-scoreboard to its first goal: A replacement of the scoreboard by normal shared memory. To reach this the experiments of health_checker should be removed or changed to a provider of bytraffic/byrequest for mod_proxy_balancer. Any comments? Cheers

Re: svn commit: r550519 - in /httpd/httpd/trunk: docs/manual/mod/mod_proxy.html.en docs/manual/mod/mod_proxy.xml modules/proxy/mod_proxy.c modules/proxy/mod_proxy.h modules/proxy/mod_proxy_balancer.c

2007-06-26 Thread jean-frederic clere
Ruediger Pluem wrote: Ok, partly playing a bit of devils advocate below :-). On 06/25/2007 04:42 PM, [EMAIL PROTECTED] wrote: Author: jfclere Date: Mon Jun 25 07:42:25 2007 New Revision: 550519 URL: http://svn.apache.org/viewvc?view=revrev=550519 Log: Add sticky_path to solve PR41897.

Re: svn commit: r550519 - in /httpd/httpd/trunk: docs/manual/mod/mod_proxy.html.en docs/manual/mod/mod_proxy.xml modules/proxy/mod_proxy.c modules/proxy/mod_proxy.h modules/proxy/mod_proxy_balancer.c

2007-06-27 Thread jean-frederic clere
Ruediger Pluem wrote: On 06/26/2007 06:46 PM, jean-frederic clere wrote: Ruediger Pluem wrote: Ok, furthermore I think we need to adjust the proxy_status_hook to actually display the string the user configured and not only the path for the cookie. The same is true for the balancer

Re: svn commit: r551099 - in /httpd/httpd/trunk/modules/proxy: mod_proxy.c mod_proxy_balancer.c

2007-06-27 Thread jean-frederic clere
Ruediger Pluem wrote: On 06/27/2007 10:42 AM, [EMAIL PROTECTED] wrote: Author: jfclere Date: Wed Jun 27 01:42:02 2007 New Revision: 551099 URL: http://svn.apache.org/viewvc?view=revrev=551099 Log: Remove the balancer modification logic of balancer-manager because it can't work. And add the

apr_table_setn in mod_proxy_balancer.c

2007-06-27 Thread jean-frederic clere
Hi, In mod_proxy_balancer.c there is the following code: +++ /* Add the session route to request notes if present */ if (route) { apr_table_setn(r-notes, session-sticky, (*balancer)-sticky); apr_table_setn(r-notes, session-route, route); /* Add session info to env. */

Re: svn commit: r551171 - /httpd/httpd/branches/2.2.x/STATUS

2007-06-28 Thread jean-frederic clere
Jim Jagielski wrote: On Jun 27, 2007, at 11:08 AM, Nick Kew wrote: On Wed, 27 Jun 2007 14:17:36 - [EMAIL PROTECTED] wrote: +* mod_proxy: Arrange the timeout handling. + Trunk version of patch: +http://svn.apache.org/viewvc?view=revrevision=550514 +

Re: apr_table_setn in mod_proxy_balancer.c

2007-06-28 Thread jean-frederic clere
Ruediger Pluem wrote: On 06/27/2007 05:09 PM, jean-frederic clere wrote: Hi, In mod_proxy_balancer.c there is the following code: +++ /* Add the session route to request notes if present */ if (route) { apr_table_setn(r-notes, session-sticky, (*balancer)-sticky

Re: svn commit: r551171 - /httpd/httpd/branches/2.2.x/STATUS

2007-06-28 Thread jean-frederic clere
Ruediger Pluem wrote: On 06/27/2007 05:51 PM, Jim Jagielski wrote: On Jun 27, 2007, at 11:08 AM, Nick Kew wrote: On Wed, 27 Jun 2007 14:17:36 - [EMAIL PROTECTED] wrote: +* mod_proxy: Arrange the timeout handling. + Trunk version of patch: +

Re: svn commit: r551171 - /httpd/httpd/branches/2.2.x/STATUS

2007-06-28 Thread jean-frederic clere
Nick Kew wrote: On Wed, 27 Jun 2007 14:17:36 - [EMAIL PROTECTED] wrote: +* mod_proxy: Arrange the timeout handling. + Trunk version of patch: +http://svn.apache.org/viewvc?view=revrevision=550514 +http://svn.apache.org/viewvc?view=revrevision=546128 + +1:

Re: svn commit: r551171 - /httpd/httpd/branches/2.2.x/STATUS

2007-06-28 Thread jean-frederic clere
Plüm wrote: -Ursprüngliche Nachricht- Von: jean-frederic clere Gesendet: Donnerstag, 28. Juni 2007 09:11 An: dev@httpd.apache.org Betreff: Re: svn commit: r551171 - /httpd/httpd/branches/2.2.x/STATUS Nick Kew wrote: On Wed, 27 Jun 2007 14:17:36 - [EMAIL PROTECTED] wrote

Re: apr_table_setn in mod_proxy_balancer.c

2007-07-09 Thread jean-frederic clere
Ruediger Pluem wrote: On 06/27/2007 05:09 PM, jean-frederic clere wrote: Hi, In mod_proxy_balancer.c there is the following code: +++ /* Add the session route to request notes if present */ if (route) { apr_table_setn(r-notes, session-sticky, (*balancer)-sticky

Re: apr_table_setn in mod_proxy_balancer.c

2007-07-09 Thread jean-frederic clere
jean-frederic clere wrote: Ruediger Pluem wrote: On 06/27/2007 05:09 PM, jean-frederic clere wrote: Hi, In mod_proxy_balancer.c there is the following code: +++ /* Add the session route to request notes if present */ if (route) { apr_table_setn(r-notes, session-sticky

httpd returns 500 when a route does not belong to the balancer.

2007-07-12 Thread jean-frederic clere
Hi, I would like to fix a problem in mod_proxy_balancer when the configuration is something like: +++ Location /myapp ProxyPass balancer://mycluster/myapp stickysession=JESSSIONID|jsessionid nofailover=On /Location Location /titi ProxyPass balancer://mytiti/titi

Re: httpd returns 500 when a route does not belong to the balancer.

2007-07-12 Thread jean-frederic clere
Plüm wrote: -Ursprüngliche Nachricht- Von: jean-frederic clere Gesendet: Donnerstag, 12. Juli 2007 13:37 An: dev@httpd.apache.org Betreff: httpd returns 500 when a route does not belong to the balancer. Hi, I would like to fix a problem in mod_proxy_balancer when

Re: httpd returns 500 when a route does not belong to the balancer.

2007-07-12 Thread jean-frederic clere
Mladen Turk wrote: jean-frederic clere wrote: Hi, I would like to fix a problem in mod_proxy_balancer when the configuration is something like: +++ Location /myapp ProxyPass balancer://mycluster/myapp stickysession=JESSSIONID|jsessionid nofailover=On /Location Location /titi

Re: httpd returns 500 when a route does not belong to the balancer.

2007-07-12 Thread jean-frederic clere
Mladen Turk wrote: jean-frederic clere wrote: Mladen Turk wrote: jean-frederic clere wrote: Hi, I would like to fix a problem in mod_proxy_balancer when the configuration is something like: +++ Location /myapp ProxyPass balancer://mycluster/myapp stickysession=JESSSIONID|jsessionid

Re: httpd returns 500 when a route does not belong to the balancer.

2007-07-17 Thread jean-frederic clere
Mladen Turk wrote: jean-frederic clere wrote: Hi, I would like to fix a problem in mod_proxy_balancer when the configuration is something like: +++ Location /myapp ProxyPass balancer://mycluster/myapp stickysession=JESSSIONID|jsessionid nofailover=On /Location Location /titi

Re: svn commit: r556931 - /httpd/httpd/trunk/modules/proxy/mod_proxy_balancer.c

2007-07-17 Thread jean-frederic clere
Jim Jagielski wrote: On Jul 17, 2007, at 10:31 AM, [EMAIL PROTECTED] wrote: +if (*(workers-s-route) strcmp(workers-s-route, route) == 0) { Is that right? I'm guessing the 1st check was to make sure that workers-s-route wasn't NULL (and therefore the strcmp didn't dump), but

Re: 2.2.5?

2007-07-19 Thread jean-frederic clere
Ruediger Pluem wrote: Now that the security related patches have been backported to 2.2.x is there anything that prevents us from releasing 2.2.5? for me, not really I just want to finish the mod_proxy stuff related to PR37770 to get it in this release. Cheers Jean-Frederic Sander Temme

docs/manual/mod/mod_proxy

2007-07-20 Thread jean-frederic clere
Hi, What about removing the +++ Proxy * Order deny,allow Allow from all /Proxy +++ In the proxy reserve example? Cheers Jean-Frederic Index: docs/manual/mod/mod_proxy.xml === --- docs/manual/mod/mod_proxy.xml (revision

Re: [PATCH] proxy/ajp_header.c: Fix header detection

2007-08-30 Thread jean-frederic clere
Martin Kraemer wrote: Hi. While looking at ajp_header.c, I realized that its method of parsing the header line tokens is flakey: it uses memcmp() to check, e.g., whether the header token is Accept-Charset:, by uppercasing the token name (- ACCEPT-CHARSET), then compares the initial ACCEPT-

Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h

2007-09-07 Thread jean-frederic clere
Martin Kraemer wrote: On Fri, Sep 07, 2007 at 10:30:13AM +0200, jean-frederic clere wrote: That is going to break: In mod_proxy.h: int ap_proxy_lb_workers(void); In server/scoreboard.c: +++ static APR_OPTIONAL_FN_TYPE(ap_proxy_lb_workers) *proxy_lb_workers

Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h

2007-09-07 Thread jean-frederic clere
Martin Kraemer wrote: On Fri, Sep 07, 2007 at 10:30:13AM +0200, jean-frederic clere wrote: That is going to break: In mod_proxy.h: int ap_proxy_lb_workers(void); In server/scoreboard.c: +++ static APR_OPTIONAL_FN_TYPE(ap_proxy_lb_workers) *proxy_lb_workers

Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h

2007-09-07 Thread jean-frederic clere
That is going to break: In mod_proxy.h: int ap_proxy_lb_workers(void); In server/scoreboard.c: +++ static APR_OPTIONAL_FN_TYPE(ap_proxy_lb_workers) *proxy_lb_workers; +++ Cheers Jean-Frederic [EMAIL PROTECTED] wrote: Author: jim Date: Thu Sep 6 06:46:46 2007

Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h

2007-09-10 Thread jean-frederic clere
Jim Jagielski wrote: Ruediger Pluem wrote: Because it was int ap_proxy_lb_workers(void); and not PROXY_DECLARE(int)ap_proxy_lb_workers(void); Yep. Ideally, of course, a design like mod_ssl with the ssl_private.h usage is nice, because there you can expose functions that need to be

Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h

2007-09-10 Thread jean-frederic clere
Plüm wrote: -Ursprüngliche Nachricht- Von: jean-frederic clere [mailto:[EMAIL PROTECTED] Gesendet: Montag, 10. September 2007 13:23 An: dev@httpd.apache.org Cc: Jim Jagielski Betreff: Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h Plüm wrote

Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h

2007-09-10 Thread jean-frederic clere
Jim Jagielski wrote: On Sep 10, 2007, at 6:37 AM, Plüm, Rüdiger, VF-Group wrote: For example what about adding: static APR_OPTIONAL_FN_TYPE(ap_proxy_lb_worker_size) *proxy_lb_worker_size; and use a void * in scoreboard and an int for the size? For me this sounds fine, but I would guess

Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h

2007-09-10 Thread jean-frederic clere
Jim Jagielski wrote: On Sep 10, 2007, at 8:24 AM, jean-frederic clere wrote: Jim Jagielski wrote: On Sep 10, 2007, at 6:37 AM, Plüm, Rüdiger, VF-Group wrote: For example what about adding: static APR_OPTIONAL_FN_TYPE(ap_proxy_lb_worker_size) *proxy_lb_worker_size; and use a void

Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h

2007-09-10 Thread jean-frederic clere
Jim Jagielski wrote: On Sep 10, 2007, at 6:37 AM, Plüm, Rüdiger, VF-Group wrote: For example what about adding: static APR_OPTIONAL_FN_TYPE(ap_proxy_lb_worker_size) *proxy_lb_worker_size; and use a void * in scoreboard and an int for the size? For me this sounds fine, but I would guess

Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h

2007-09-10 Thread jean-frederic clere
Plüm wrote: -Ursprüngliche Nachricht- Von: jean-frederic clere Gesendet: Montag, 10. September 2007 16:38 An: dev@httpd.apache.org Betreff: Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h Jim Jagielski wrote: On Sep 10, 2007, at 6:37 AM, Plüm, Rüdiger, VF

Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h

2007-09-10 Thread jean-frederic clere
Jim Jagielski wrote: On Sep 10, 2007, at 12:19 PM, jean-frederic clere wrote: Well that just because I am thinking that the size of proxy_worker_stat could be changed by a balancer, like adding extra bytes to proxy_worker_stat. That is what 'void *context' is designed to do

Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h

2007-09-11 Thread jean-frederic clere
Plüm wrote: -Ursprüngliche Nachricht- Von: jean-frederic clere Gesendet: Montag, 10. September 2007 16:38 An: dev@httpd.apache.org Betreff: Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h Jim Jagielski wrote: On Sep 10, 2007, at 6:37 AM, Plüm, Rüdiger, VF

Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h

2007-09-11 Thread jean-frederic clere
Jim Jagielski wrote: On Sep 11, 2007, at 12:09 PM, jean-frederic clere wrote: 1. IMHO requires a minor bump. Find a patch that covers all the points you raised below. More comments? Requires a major bump. Also destroys all those mystical other balancers from working as-is, since

Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h

2007-09-12 Thread jean-frederic clere
Ruediger Pluem wrote: On 09/11/2007 06:09 PM, jean-frederic clere wrote: Plüm wrote: -Ursprüngliche Nachricht- Von: jean-frederic clere Gesendet: Montag, 10. September 2007 16:38 An: dev@httpd.apache.org Betreff: Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h

Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h

2007-09-12 Thread jean-frederic clere
Ruediger Pluem wrote: On 09/11/2007 10:51 PM, Jim Jagielski wrote: I would also suggest that we keep ap_proxy_lb_workers in proxy_util.c (as it currently is), since even though its not part of the API, other proxy module make have a need for it (and again, keeping the number of changes

Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h

2007-09-12 Thread jean-frederic clere
Plüm wrote: -Ursprüngliche Nachricht- Von: jean-frederic clere Gesendet: Mittwoch, 12. September 2007 10:34 An: dev@httpd.apache.org Betreff: Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h Ruediger Pluem wrote: On 09/11/2007 10:51 PM, Jim Jagielski wrote

Re: svn commit: r573264 - /httpd/httpd/trunk/include/scoreboard.h

2007-09-12 Thread jean-frederic clere
Jim Jagielski wrote: On Sep 11, 2007, at 12:44 PM, jean-frederic clere wrote: Jim Jagielski wrote: On Sep 11, 2007, at 12:09 PM, jean-frederic clere wrote: 1. IMHO requires a minor bump. Find a patch that covers all the points you raised below. More comments? Requires a major

mod_proxy_ajp sockets in CLOSE_WAIT state

2007-10-31 Thread jean-frederic clere
Hi, After running a load test (using ab) against Apache httpd + Tomcat I have noted several sockets are left in CLOSE_WAIT state when using MPM worker. Once the worker is marked IN_ERROR the sockets belonging to the pool will not be closed. I have tried something like the worker.maintain of

Re: memory leak in 2.2.x balancer?

2007-11-27 Thread jean-frederic clere
Jeff Trawick wrote: On Nov 27, 2007 8:47 AM, Plüm, Rüdiger, VF-Group [EMAIL PROTECTED] wrote: -Ursprüngliche Nachricht- Von: Jeff Trawick Gesendet: Dienstag, 27. November 2007 14:21 An: dev@httpd.apache.org Betreff: memory leak in 2.2.x balancer? looks like a leak to me; what do

Re: memory leak in 2.2.x balancer?

2007-11-28 Thread jean-frederic clere
Jim Jagielski wrote: On Nov 27, 2007, at 10:16 AM, jean-frederic clere wrote: Jeff Trawick wrote: On Nov 27, 2007 8:47 AM, Plüm, Rüdiger, VF-Group [EMAIL PROTECTED] wrote: -Ursprüngliche Nachricht- Von: Jeff Trawick Gesendet: Dienstag, 27. November 2007 14:21 An: dev

Re: mod_proxy in Apache 2.2 and HTTP_INTERNAL_SERVER_ERROR

2007-12-14 Thread jean-frederic clere
Jiri Tulach - Position wrote: It's possible fault of mod_fastcgi which returns HTTP_INTERNAL_SERVER_ERROR in cases when server is overloaded or external fastcgi instances are unavailable. Unfortunately mod_fastcgi is not actively developed even thought lot of people use it in production

modules/aaa/mod_authz_owner.c

2006-01-19 Thread Jean-frederic Clere
Hi, On ReliantUnix the following code can't be compiled: +++ typedef struct { } authz_owner_config_rec; +++ Because the structure is empty. Any problem to apply the following patch: +++ Index: aaa/mod_authz_owner.c === ---

Re: [PATCH 1/6] scoreboard over-sized

2006-05-16 Thread Jean-frederic Clere
Jim Jagielski wrote: Chris Darroch wrote: Personally, if I was writing a completely private module, I'd probably just patch the scoreboard.h definition of lb_score to include whatever extra stuff I needed, and recompile httpd. That way I'm insulated from any changes to the size of either

Re: Compiling a C++ module with g++ on Solaris

2006-06-12 Thread Jean-frederic Clere
Phil Endecott wrote: Dear All, I'm the author of Anyterm (http://anyterm.org/) which uses an Apache module written in C++. This works OK on Linux. However, a user has attempted to compile it on Solaris and it fails at run time. Specifically, the first time that any C++ memory allocation

Re: mod_proxy_balancer/mod_proxy_ajp TODO

2006-06-22 Thread Jean-frederic Clere
Henri Gomez wrote: The TomcatoMips indicator was just something to tell that it's not the raw CPU power which is important, but the estimated LOAD capacity of an instance. Accounting informations should included average time to execute a request, number of thread in use (AJP/HTTP), estimated

proxy_worker_stat in mod_proxy

2006-07-08 Thread Jean-frederic Clere
Hi, I am looking in mod_proxy and I think that proxy_worker_stat of proxy_worker should be replaced by some void * to allow to write new balancers. The idea is each worker needs to point to the view the balancer of its, but only the balancer needs to know the structure it needs to make the

Re: [Fwd: Re: proxy_worker_stat in mod_proxy]

2006-07-10 Thread Jean-frederic Clere
Ruediger Pluem wrote: On 09.07.2006 16:37, Jim Jagielski wrote: Ruediger Pluem wrote: So it makes sense from my perspective to add a void pointer *to* the proxy_worker_stat structure to enable balancers to store custom data that is private to them. This already exists. See

Re: [Fwd: Re: proxy_worker_stat in mod_proxy]

2006-07-11 Thread Jean-frederic Clere
Ruediger Pluem wrote: On 07/10/2006 12:21 PM, Jean-frederic Clere wrote: Ruediger Pluem wrote: context could be used. But I am thinking of sharing the balacing information with another program. By changing proxy_worker_stat * to a void * and arranging the corresponding code the first step

ap_proxy_get_worker

2006-07-12 Thread Jean-frederic Clere
Hi, Why does ap_proxy_get_worker() gives the best matched worker? - Shouldn't it give the exact match - Cheers Jean-Frederic

Re: AW: ap_proxy_get_worker

2006-07-12 Thread Jean-frederic Clere
Plüm wrote: -Ursprüngliche Nachricht- Von: Jean-frederic Clere Gesendet: Mittwoch, 12. Juli 2006 14:21 An: dev@httpd.apache.org Betreff: ap_proxy_get_worker Hi, Why does ap_proxy_get_worker() gives the best matched worker? - Shouldn't it give the exact match - Normally

Additing a storage for the shared information of the worker in mod_proxy

2006-07-12 Thread Jean-frederic Clere
Hi, I am still trying to replace the scoreboard by shared memory to store the shared information of the workers, I am now thinking to get this by adding modules like the prototype I have enclosed (that is a patch against trunk). Does this look to be the right way to go? Or has someone a

Re: Additing a storage for the shared information of the worker in mod_proxy

2006-07-12 Thread Jean-frederic Clere
Brian Akins wrote: Jean-frederic Clere wrote: Hi, I am still trying to replace the scoreboard by shared memory to store the shared information of the workers, I am now thinking to get this by adding modules like the prototype I have enclosed (that is a patch against trunk). We really need

Re: Additing a storage for the shared information of the worker in

2006-07-13 Thread Jean-frederic Clere
Jim Jagielski wrote: Brian Akins wrote: Jim Jagielski wrote: +1. For example, a memcached based scoreboard would be pretty cool ;) maybe in mod_scoreboard it may use a provider mechanism to actually implement the scoreboard. Maybe have an ap_scoreboard_create_ex where you

Re: Additing a storage for the shared information of the worker in mod_proxy

2006-07-13 Thread Jean-frederic Clere
Brian Akins wrote: Jim Jagielski wrote: If this is data that needs to be accessed from non-proxy modules then yes, I agree. A basic API could look like. By worker, I am thinking about the mpm sense, not the proxy sense. I guess slot may be a better term: /*used for ap_scoreboard_do.

Re: Additing a storage for the shared information of the worker in mod_proxy

2006-07-13 Thread Jean-frederic Clere
Brian Akins wrote: Jean-frederic Clere wrote: With such an interface you assume only one process will access to one slot... That is what the scoreboard allows. Allowing updates from different proccesses on the same slot. Should we have an ap_slot_read_look() and an ap_slot_unlock

Re: Additing a storage for the shared information of the worker in mod_proxy

2006-07-13 Thread Jean-frederic Clere
Brian Akins wrote: Jim Jagielski wrote: Having some external (or even internal) process update a slot that isn't its own is dangerous. And the required locking would be slow. In my own hacked proxy, an external healthchecker and the proxy share a piece of shared memory that is read-only

Re: Additing a storage for the shared information of the worker in mod_proxy

2006-07-13 Thread Jean-frederic Clere
Brian Akins wrote: Jim Jagielski wrote: If this is data that needs to be accessed from non-proxy modules then yes, I agree. A basic API could look like. By worker, I am thinking about the mpm sense, not the proxy sense. I guess slot may be a better term: /*used for ap_scoreboard_do.

Re: Additing a storage for the shared information of the worker in

2006-07-14 Thread Jean-frederic Clere
Jim Jagielski wrote: Jean-frederic Clere wrote: Using the result of your ideas and the explaintions now I have mod_proxy that uses the scoreboard via a scoreboard provider ;-) Find enclosed the code. Just one comment off the top of my head: this seems to confuse what is the actual

Re: Additing a storage for the shared information of the worker in

2006-07-14 Thread Jean-frederic Clere
Brian Akins wrote: Jim Jagielski wrote: at is the actual scoreboard and what is the proxy's segment of scoreboard space... Eg the ap_scoreboard_* stuff implies that it's Apache's real scoreboard. Like I said, I think this scoreboard stuff should be more generic than just proxy. It could

mem memory providers

2006-07-14 Thread Jean-frederic Clere
Hi, I am starting an thread that could go under ap_proxy_get_worker and proxy_worker_stat in mod_proxy ;-) Because I would like to create a new family of modules under mem (httpd/modules/mem): shared memory provider. The idea is to have modules that provides memory that could used by slots

Re: mod_proxy feature coming up

2006-07-18 Thread Jean-frederic Clere
Jim Jagielski wrote: Pretty soon I'll be committing my balancer set patch to httpd-trunk. This basically allows for member sets within a balancer similar to the 'distance' mod_jk attribute. What it does is allow for more control over which members will be used via m_p_b. The logic is:

Re: mod_proxy feature coming up

2006-07-18 Thread Jean-frederic Clere
Jim Jagielski wrote: Jean-frederic Clere wrote: Jim Jagielski wrote: Pretty soon I'll be committing my balancer set patch to httpd-trunk. This basically allows for member sets within a balancer similar to the 'distance' mod_jk attribute. What it does is allow for more control over

Re: mod_proxy feature coming up

2006-07-18 Thread Jean-frederic Clere
Jim Jagielski wrote: On Jul 18, 2006, at 11:29 AM, Jim Jagielski wrote: Yes, I'd propose waiting to commit that. The sole reason is that the member-set and other previously committed patches will likely be more readily approved for backporting to 2.2.x, whereas the scoreboard changes might

Re: svn commit: r423444 - in /httpd/httpd/branches/httpd-proxy-scoreboard/modules/mem: ./ Makefile.in config5.m4 mod_plainmem.c mod_scoreboard.c mod_sharedmem.c slotmem.h

2006-07-20 Thread Jean-frederic Clere
Ruediger Pluem wrote: Some comments inline. Regards Many thanks Cheers Jean-Frederic Rüdiger On 19.07.2006 14:18, [EMAIL PROTECTED] wrote: Author: jfclere Date: Wed Jul 19 05:18:10 2006 New Revision: 423444 ---

Re: svn commit: r423444 - in /httpd/httpd/branches/httpd-proxy-scoreboard/modules/mem: ./ Makefile.in config5.m4 mod_plainmem.c mod_scoreboard.c mod_sharedmem.c slotmem.h

2006-07-21 Thread Jean-frederic Clere
Brian Akins wrote: Jean-frederic Clere wrote: Do you mean the proxy back-end connections? I am thinking of a more general purpose slotmem not particularly tied to proxy. Maybe have some wrapper functions that create a slotmem based on threads x procs and can be access using r

Re: Balancer manager: bug and feature request

2006-07-24 Thread Jean-frederic Clere
Andrew Stribblehill wrote: That is, bug and feature request, not bug and feature request. As I reported in http://issues.apache.org/bugzilla/show_bug.cgi?id=39907 I'm finding that with the worker MPM and a pristine httpd 2.2.2 on an i386 Debian platform, changes I make to the configuration

  1   2   3   4   5   >