Re: Segfault in ssl_scache_init on reload

2016-10-09 Thread Ruediger Pluem
On 10/08/2016 09:28 PM, Niklas Edmundsson wrote: > > Hi all. > > httpd 2.4.23, built from httpd-2.4.23.tar.bz2 and httpd-2.4.23-deps.tar.bz2 > on Ubuntu 14.04.5 LTS (trusty). > > While fiddling with enabling https on ftp.acc.umu.se I stumbled on the > following coredump when reloading the se

Re: Redis and mod_cache/mod_socache

2016-10-31 Thread Ruediger Pluem
On 10/31/2016 02:43 PM, Jim Jagielski wrote: > It would, but that would mean even more of a APR dependency > and a wait until the next release of APR and etc, etc, etc... > Basically, APR moves too slow for httpd. Which creates the generic question for me: Can't we setup APR in a way that for c

Re: svn commit: r1767482 - in /httpd/httpd/branches/2.4.x: ./ CHANGES STATUS modules/http/http_request.c server/request.c

2016-11-02 Thread Ruediger Pluem
On 11/01/2016 12:53 PM, j...@apache.org wrote: > Author: jim > Date: Tue Nov 1 11:53:57 2016 > New Revision: 1767482 > > URL: http://svn.apache.org/viewvc?rev=1767482&view=rev > Log: > Merge r1698239 from trunk: > Submitted by: covener > Reviewed/backported by: jim > > Modified: > httpd/ht

Re: svn commit: r1768036 - in /httpd/httpd/branches/2.4.x-merge-http-strict: ./ CHANGES include/ap_mmn.h include/http_core.h include/httpd.h modules/http/http_filters.c server/core.c server/protocol.c

2016-11-08 Thread Ruediger Pluem
On 11/04/2016 03:20 PM, wr...@apache.org wrote: > Author: wrowe > Date: Fri Nov 4 14:20:16 2016 > New Revision: 1768036 > > URL: http://svn.apache.org/viewvc?rev=1768036&view=rev > Log: > Add an option to enforce stricter HTTP conformance > > This is a first stab, the checks will likely have t

Re: svn commit: r1768036 - in /httpd/httpd/branches/2.4.x-merge-http-strict: ./ CHANGES include/ap_mmn.h include/http_core.h include/httpd.h modules/http/http_filters.c server/core.c server/protocol.c

2016-11-16 Thread Ruediger Pluem
On 11/16/2016 01:08 PM, William A Rowe Jr wrote: > On Tue, Nov 8, 2016 at 1:39 PM, Ruediger Pluem <mailto:rpl...@apache.org>> wrote: > > > On 11/04/2016 03:20 PM, wr...@apache.org <mailto:wr...@apache.org> wrote: > > Author: wrowe > > Dat

Re: svn commit: r1769965 - /httpd/httpd/trunk/server/vhost.c

2016-11-16 Thread Ruediger Pluem
On 11/16/2016 01:05 PM, wr...@apache.org wrote: > Author: wrowe > Date: Wed Nov 16 12:05:53 2016 > New Revision: 1769965 > > URL: http://svn.apache.org/viewvc?rev=1769965&view=rev > Log: > Actually cause the Host header to be overridden, as noted by rpluem, > and simplify now that there isn't a

Re: bug with SSLVerifyClient?

2016-11-23 Thread Ruediger Pluem
On 11/23/2016 07:55 PM, Helmut K. C. Tessarek wrote: > On 2016-11-23 13:43, Eric Covener wrote: >> In your desired config, the initial handshake happens with >> SSLVerifyClient=none, so no client certificate is requested so none >> can be sent by the client. >> The initial handshake completes, th

Re: svn commit: r1772678 [2/2] - in /httpd/httpd/branches/2.4.x: ./ docs/manual/mod/ include/ modules/http/ server/

2016-12-07 Thread Ruediger Pluem
Sorry for chiming in so late :-(. On 12/05/2016 03:34 PM, j...@apache.org wrote: > Modified: httpd/httpd/branches/2.4.x/server/protocol.c > URL: > http://svn.apache.org/viewvc/httpd/httpd/branches/2.4.x/server/protocol.c?rev=1772678&r1=1772677&r2=1772678&view=diff > ==

Re: svn commit: r1772678 [1/2] - in /httpd/httpd/branches/2.4.x: ./ docs/manual/mod/ include/ modules/http/ server/

2016-12-07 Thread Ruediger Pluem
Sorry for chiming in so late :-(. On 12/05/2016 03:34 PM, j...@apache.org wrote: > Author: jim > Date: Mon Dec 5 14:34:29 2016 > New Revision: 1772678 > > URL: http://svn.apache.org/viewvc?rev=1772678&view=rev > Log: > Modified: httpd/httpd/branches/2.4.x/modules/http/http_filters.c > URL: >

Re: svn commit: r1773865 - /httpd/httpd/trunk/modules/http/http_filters.c

2016-12-13 Thread Ruediger Pluem
On 12/13/2016 02:49 PM, Yann Ylavic wrote: > On Tue, Dec 13, 2016 at 10:48 AM, Plüm, Rüdiger, Vodafone Group > wrote: >> Another aspect of all these patches that I don't get is why we need >> to eat the contents of the original brigade? IMHO we don't need to do >> this. It is thrown away automa

Re: svn commit: r1773865 - /httpd/httpd/trunk/modules/http/http_filters.c

2016-12-13 Thread Ruediger Pluem
On 12/13/2016 08:57 PM, Jacob Champion wrote: > On 12/13/2016 11:37 AM, Ruediger Pluem wrote: >> On 12/13/2016 02:49 PM, Yann Ylavic wrote: >>> I don't find the change too complex after all, and that's a quite >>> critical filter for doing the right/safe thi

Re: T&R of 2.4.24

2016-12-15 Thread Ruediger Pluem
On 12/15/2016 08:09 PM, Eric Covener wrote: > On Thu, Dec 15, 2016 at 10:16 AM, Eric Covener wrote: >> On Thu, Dec 15, 2016 at 10:13 AM, Jim Jagielski wrote: >>> From what I can see, there are no show-stoppers and >>> all my tests show no regressions... >>> >>> Let's shoot for a T&R this (east

Re: AW: svn commit: r1773865 - /httpd/httpd/trunk/modules/http/http_filters.c

2016-12-19 Thread Ruediger Pluem
On 12/15/2016 11:13 AM, Plüm, Rüdiger, Vodafone Group wrote: > > >> -Ursprüngliche Nachricht- >> Von: Yann Ylavic [mailto:ylavic@gmail.com] >> Gesendet: Mittwoch, 14. Dezember 2016 19:58 >> An: httpd-dev >> Betreff: Re: svn commit: r1773865 - >> /httpd/httpd/trunk/modules/http/http

Re: svn commit: r1774328 - in /httpd/test/framework/trunk: c-modules/test_utilities/ c-modules/test_utilities/mod_test_utilities.c t/conf/extra.conf.in t/modules/rewrite.t

2016-12-19 Thread Ruediger Pluem
On 12/14/2016 08:37 PM, jchamp...@apache.org wrote: > Author: jchampion > Date: Wed Dec 14 19:37:03 2016 > New Revision: 1774328 > > URL: http://svn.apache.org/viewvc?rev=1774328&view=rev > Log: > PR60478: add test case > > Includes a utility ap_expr function to determine the length of a string

Re: svn commit: r1776285 - /httpd/httpd/trunk/modules/proxy/mod_proxy_wstunnel.c

2017-01-10 Thread Ruediger Pluem
On 12/28/2016 03:40 PM, cove...@apache.org wrote: > Author: covener > Date: Wed Dec 28 14:40:54 2016 > New Revision: 1776285 > > URL: http://svn.apache.org/viewvc?rev=1776285&view=rev > Log: > improve a debug message > > > Modified: > httpd/httpd/trunk/modules/proxy/mod_proxy_wstunnel.c >

Re: svn commit: r1776575 - in /httpd/httpd/trunk: docs/log-message-tags/next-number docs/manual/mod/mod_remoteip.xml modules/metadata/mod_remoteip.c

2017-01-10 Thread Ruediger Pluem
On 12/30/2016 03:20 PM, drugg...@apache.org wrote: > Author: druggeri > Date: Fri Dec 30 14:20:48 2016 > New Revision: 1776575 > > URL: http://svn.apache.org/viewvc?rev=1776575&view=rev > Log: > Merge new PROXY protocol code into mod_remoteip > > Modified: > httpd/httpd/trunk/docs/log-messa

Re: svn commit: r1776575 - in /httpd/httpd/trunk: docs/log-message-tags/next-number docs/manual/mod/mod_remoteip.xml modules/metadata/mod_remoteip.c

2017-01-16 Thread Ruediger Pluem
Anyone? Regards Rüdiger On 01/10/2017 12:39 PM, Ruediger Pluem wrote: > > > On 12/30/2016 03:20 PM, drugg...@apache.org wrote: >> Author: druggeri >> Date: Fri Dec 30 14:20:48 2016 >> New Revision: 1776575 >> >> URL: http://svn.apache.org/viewvc?rev=1776

Re: clang-analyzer?

2017-01-19 Thread Ruediger Pluem
On 01/19/2017 08:16 PM, William A Rowe Jr wrote: > On Mon, Jan 9, 2017 at 3:48 AM, Graham Leggett wrote: >> On 08 Jan 2017, at 4:45 AM, Leif Hedstrom wrote: >> >>> I ran clang-analyzer against the HTTPD master branch, and it found 126 >>> issues. Many of these are benign, but I was curious if

Re: clang-analyzer?

2017-01-19 Thread Ruediger Pluem
On 01/19/2017 09:07 PM, William A Rowe Jr wrote: > On Thu, Jan 19, 2017 at 1:30 PM, Ruediger Pluem wrote: >> >> If they are no-ops as you state in 3. how could they introduce regressions? > > They are still a text and code change. Cleaning up a cast, for example may &

Re: svn commit: r1779525 - /httpd/httpd/trunk/modules/http2/h2_mplx.c

2017-01-19 Thread Ruediger Pluem
On 01/19/2017 09:38 PM, ic...@apache.org wrote: > Author: icing > Date: Thu Jan 19 20:38:50 2017 > New Revision: 1779525 > > URL: http://svn.apache.org/viewvc?rev=1779525&view=rev > Log: > On the trunk: > > mod_http2: decoupling lifetime of mplx pool from h2_session which messed up > the clean

Re: Reset out x.minor.z definition of 'minor' at httpd?

2017-01-20 Thread Ruediger Pluem
On 01/20/2017 09:37 PM, Graham Leggett wrote: > On 20 Jan 2017, at 7:47 PM, David Zuelke wrote: > >> I'd actually like to question the whole practice of porting features back to >> older branches. I think that's the core reason why trunk is in total >> disarray, and why no substantial release

Re: svn commit: r1779700 - /httpd/httpd/trunk/modules/filters/mod_brotli.c

2017-01-23 Thread Ruediger Pluem
On 01/21/2017 09:57 AM, Yann Ylavic wrote: > Hi, > > On Sat, Jan 21, 2017 at 7:40 AM, wrote: >> Author: jailletc36 >> Date: Sat Jan 21 06:40:23 2017 >> New Revision: 1779700 >> >> URL: http://svn.apache.org/viewvc?rev=1779700&view=rev >> Log: >> Save a few bytes and a few cycles. >> >> Modifie

Re: svn commit: r1776575 - in /httpd/httpd/trunk: docs/log-message-tags/next-number docs/manual/mod/mod_remoteip.xml modules/metadata/mod_remoteip.c

2017-01-24 Thread Ruediger Pluem
On 01/17/2017 02:48 PM, Jim Jagielski wrote: > >> On Jan 16, 2017, at 6:57 PM, Daniel Ruggeri wrote: >> >> For the most part, yes except the portions that make the header presence >> optional (the HDR_MISSING case). Those were added as it came into the >> code base to handle a use case I was wo

Re: svn commit: r425454 - in /httpd/httpd/trunk: CHANGES modules/proxy/NWGNUproxyajp modules/proxy/ajp_utils.c modules/proxy/mod_proxy.c modules/proxy/mod_proxy.h modules/proxy/mod_proxy_ajp.c modules

2006-07-25 Thread Ruediger Pluem
On 07/25/2006 06:50 PM, [EMAIL PROTECTED] wrote: > Author: mturk > Date: Tue Jul 25 09:50:07 2006 > New Revision: 425454 > > URL: http://svn.apache.org/viewvc?rev=425454&view=rev > Log: > Added cping/cpong support for the AJP protocol. > > Added: > httpd/httpd/trunk/modules/proxy/ajp_utils.

Re: proxy balancer backports for 2.2.3

2006-07-26 Thread Ruediger Pluem
On 07/26/2006 12:07 PM, Nick Kew wrote: > On Wednesday 26 July 2006 11:02, Mladen Turk wrote: > > >>There are lots of things to backport. IMHO its the entire HEAD, >>and spread over the multiple svn commits. >>How we should deal with that? >>Having multiple backports or a single one? > > > IM

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

2006-07-26 Thread Ruediger Pluem
On 26.07.2006 15:42, [EMAIL PROTECTED] wrote: > Author: jfclere > Date: Wed Jul 26 06:42:43 2006 > New Revision: 425734 > > URL: http://svn.apache.org/viewvc?rev=425734&view=rev > Log: > Add ap_slotmem_attach() to the slotmem_storage_method. > Cut mod_sharemem.c in 2 so that its features could be

Re: 401 response with reject ip?

2006-07-26 Thread Ruediger Pluem
On Mon, Jul 24, 2006 at 9:02 AM, in message <[EMAIL PROTECTED]>, > > > >Well, I think that the following patch in mod_authz_core.c fixes the > problem that you are looking at: > > @@ -628,16 +633,25 @@ > > switch (auth_result) { > case AUTHZ_DENIED: > +

Re: 401 response with reject ip?

2006-07-26 Thread Ruediger Pluem
On 26.07.2006 17:24, Brad Nicholes wrote: > > At this point I consider it to be a bug. This is the patch that I am > currently working on that includes the use of AUTHZ_NEUTRAL return code. Then I will wait for this patch. > I think that if the reject condition is satisfied then the request >

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

2006-07-26 Thread Ruediger Pluem
On 26.07.2006 17:39, Jean-frederic Clere wrote: > Ruediger Pluem wrote: >> >> >> Too much copy and paste? We already found it above and what we are >> searching for >> is in *new isn't it? (Maybe we should also set *new to NULL in the >> beginning to &

Re: httpd-proxy-scoreboard

2006-07-26 Thread Ruediger Pluem
On 26.07.2006 18:11, Jean-frederic Clere wrote: > Hi, > > I have started to write a "generic" health-checker for mod_proxy. I > would like to change the macro PROXY_WORKER_IS_USABLE() to a routine in > proxy_util.c. > > Comments? >From my current point of view we should keep this macro as a simp

Re: Please Review! Path for os/win32/os.h - new reports Win64 when build for 64-bit

2006-07-26 Thread Ruediger Pluem
On 07/26/2006 10:10 PM, Jorge Schrauwen wrote: > Ok this is my first path so please bare with me and review. > > Since I'm rather new to C I did a lot of research and testing on this. > More information regarding the changes: > http://msdn.microsoft.com/msdnmag/issues/06/05/x64/default.aspx#S4 >

Re: httpd-proxy-scoreboard

2006-07-26 Thread Ruediger Pluem
On 07/26/2006 10:53 PM, Jean-frederic Clere wrote: > I already have a prototype of an external health checker process that > uses an AJP cping/cpong and a simple connect for http/https. > Basicaly mod_proxy uses the health_worker_method routine to write and > read from a slot mem that contains

Re: httpd-proxy-scoreboard

2006-07-27 Thread Ruediger Pluem
On 07/27/2006 11:31 AM, Jean-frederic Clere wrote: > Ruediger Pluem wrote: > >> On 07/26/2006 10:53 PM, Jean-frederic Clere wrote: >> >> >> >> >>> I already have a prototype of an external health checker process that >>> uses

Re: [VOTES] please, 2.2.3, 2.0.59, 1.3.37 releases ASAP

2006-07-27 Thread Ruediger Pluem
On 07/27/2006 08:37 PM, William A. Rowe, Jr. wrote: > [+1] httpd-2.0.59 > [+1] httpd-2.2.3 On SPARC Solaris 8 and SPARC Solaris 9 (compiled and started). Thanks for RMing. Regards Rüdiger

Re: [VOTES] please, 2.2.3, 2.0.59, 1.3.37 releases ASAP

2006-07-28 Thread Ruediger Pluem
On 07/28/2006 03:26 AM, Sander Temme wrote: > > All of these hang on the t/protocol/nntp-like.t. I'll try and put a > patch into that test that skips if we're on FreeBSD and accf_http.ko or > accf_data.ko are loaded. I am neither an expert on FreeBSD nor on accept filters, but wouldn't it be

Re: httpd-proxy-scoreboard

2006-07-28 Thread Ruediger Pluem
On 07/27/2006 02:37 PM, Jean-frederic Clere wrote: > Ruediger Pluem wrote: >> > You want to check all the connections of the pool corresponding to the > worker but not all the workers. Not all at the same time, but only the one I actually leased. > >> If it i

Re: svn commit: r426604 - in /httpd/httpd/branches/httpd-proxy-scoreboard: modules/proxy/ support/

2006-07-28 Thread Ruediger Pluem
On 28.07.2006 18:34, [EMAIL PROTECTED] wrote: > Author: jfclere > Date: Fri Jul 28 09:33:58 2006 > New Revision: 426604 > > URL: http://svn.apache.org/viewvc?rev=426604&view=rev > Log: > First try to put togother an external health checker for mod_proxy. > > > Modified: httpd/httpd/branches/htt

Re: What does the Authz "reject" directive really mean...

2006-07-28 Thread Ruediger Pluem
On 07/29/2006 12:30 AM, Brad Nicholes wrote: >There is a new concept (directive) that has been added to the > authorization (access control) portion of the web server. This new concept > is "reject". Basically what this directive does is allow you to specify > conditions by which access

Re: [PATCH] perl-framework, Skip nntp-like tests on FreeBSD with accept filters

2006-07-29 Thread Ruediger Pluem
On 07/29/2006 07:15 AM, Sander Temme wrote: > > Currently, this plan makes the tests skip on Apache 1.3 (which does not > have mod_nntp_like), Linux (where I must assume deferred accept() is Using AcceptFilter nttp none also disables deferred accepts on Linux for nttp. > enabled automatical

Re: svn commit: r426604 - in /httpd/httpd/branches/httpd-proxy-scoreboard: modules/proxy/ support/

2006-07-29 Thread Ruediger Pluem
On 07/29/2006 03:01 AM, Jean-frederic Clere wrote: > Ruediger Pluem wrote: > >> On 28.07.2006 18:34, [EMAIL PROTECTED] wrote: >> >> >>> Author: jfclere >>> Date: Fri Jul 28 09:33:58 2006 >>> New Revision: 426604 >>> >>> URL

Re: svn commit: r426604 - in /httpd/httpd/branches/httpd-proxy-scoreboard: modules/proxy/ support/

2006-07-29 Thread Ruediger Pluem
On 07/29/2006 09:59 AM, Ruediger Pluem wrote: > > But it is not used in this implementation. Instead you use worker->id here. > Should it be used by other implementations where worker->id cannot / > shouldn't be used? Never mind. Obsoleted by r426732. Regards Rüdiger

Re: svn commit: r426781 - in /httpd/httpd/branches/httpd-proxy-scoreboard: modules/mem/sharedmem_util.c support/proxymonitor.c

2006-07-29 Thread Ruediger Pluem
On 29.07.2006 16:46, Jim Jagielski wrote: > I'm not sure I'm comfortable with the directory location ("proxy") > and the names ("checker"|"comarea") being hardcoded... And > creating the directory is also somewhat unsettling as well > since it's a "normal" directory that "should" exist. It would >

Re: svn commit: r426604 - in /httpd/httpd/branches/httpd-proxy-scoreboard: modules/proxy/ support/

2006-07-29 Thread Ruediger Pluem
On 07/28/2006 06:34 PM, [EMAIL PROTECTED] wrote: > Added: > httpd/httpd/branches/httpd-proxy-scoreboard/modules/proxy/health_checker_util.c > URL: > http://svn.apache.org/viewvc/httpd/httpd/branches/httpd-proxy-scoreboard/modules/proxy/health_checker_util.c?rev=426604&view=auto > =

Re: svn commit: r427172 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy.c modules/proxy/mod_proxy.h modules/proxy/mod_proxy_balancer.c

2006-07-31 Thread Ruediger Pluem
On 07/31/2006 07:01 PM, [EMAIL PROTECTED] wrote: > Author: jim > Date: Mon Jul 31 10:01:40 2006 > New Revision: 427172 > > URL: http://svn.apache.org/viewvc?rev=427172&view=rev > Log: > Add in a very simple balancer "set" concept, which allows > for members to be assigned to a particular cluster

Re: svn commit: r427172 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy.c

2006-07-31 Thread Ruediger Pluem
On 07/31/2006 09:53 PM, Jim Jagielski wrote: > Ruediger Pluem wrote: > >> >>Shouldn't that be "while (cur_lbset <= max_lbset && !mycandidate);" >>(same question also for the other algorithm)? >>I guess otherwise we would not check for th

Re: svn commit: r427780 - in /httpd/httpd/trunk: docs/manual/mod/mod_authz_core.xml modules/aaa/mod_auth.h modules/aaa/mod_authz_core.c

2006-08-02 Thread Ruediger Pluem
On 08/02/2006 12:54 AM, [EMAIL PROTECTED] wrote: > Author: bnicholes > Date: Tue Aug 1 15:54:38 2006 > New Revision: 427780 > > URL: http://svn.apache.org/viewvc?rev=427780&view=rev > Log: > Converted the reject directive to be definitive and enabled directory_merge > to merge all of the autho

Re: svn commit: r427780 - in /httpd/httpd/trunk: docs/manual/mod/mod_authz_core.xml modules/aaa/mod_

2006-08-02 Thread Ruediger Pluem
On 08/02/2006 11:00 PM, Brad Nicholes wrote: > > > No, the default is to merge authz rules. At least that is how I understood > access control to be working by default in the past. There was no concept of > inherited authz before 2.3. Also, Joshua pointed out a flaw in my thinking > whic

Re: svn commit: r427172 - in /httpd/httpd/trunk: CHANGES modules/proxy/mod_proxy.c

2006-08-02 Thread Ruediger Pluem
On 08/01/2006 01:48 PM, Jim Jagielski wrote: > =?iso-8859-1?Q?Pl=FCm=2C_R=FCdiger=2C_VF_EITO?= wrote: > >> >> >>>-Urspr=FCngliche Nachricht- >>>Von: Ruediger Pluem=20 >>>=20 >>>BTW: Don't we need to reset checked_standby and=20

Re: svn commit: r428437 - /httpd/httpd/branches/httpd-proxy-scoreboard/modules/proxy/mod_proxy_health_checker.c

2006-08-03 Thread Ruediger Pluem
On 08/03/2006 06:01 PM, [EMAIL PROTECTED] wrote: > Author: jfclere > Date: Thu Aug 3 09:01:50 2006 > New Revision: 428437 > > URL: http://svn.apache.org/viewvc?rev=428437&view=rev > Log: > Use only one slotmem for status and checker. > > Modified: > > httpd/httpd/branches/httpd-proxy-scor

Re: [Vote] product name - for shipping versions

2006-08-10 Thread Ruediger Pluem
On 08/10/2006 09:26 PM, William A. Rowe, Jr. wrote: > William A. Rowe, Jr. wrote: > >>Roy would prefer we determine the active consensus. Please vote for what >>you believe is the principal, proper name of the product we create from >>the sources located in the http://svn.apache.org/repos/asf/h

Re: DO NOT REPLY [Bug 40234] New: - failed request with APR_INCOMPLETE

2006-08-11 Thread Ruediger Pluem
On 08/11/2006 07:12 PM, Nick Kew wrote: > On Friday 11 August 2006 17:01, [EMAIL PROTECTED] wrote: > > >>http://issues.apache.org/bugzilla/show_bug.cgi?id=40234 >> >> Summary: failed request with APR_INCOMPLETE >> Product: Apache httpd-2 >> Version: 2.3-HEAD >> >>O

Re: [PATCH 40026] ServerTokens Off

2006-08-21 Thread Ruediger Pluem
On 08/21/2006 12:34 AM, Lars Eilebrecht wrote: > > For offering such an option with Apache I've only seen two arguments: > > 1. Making the server more secure by not revealing any (or fake) >server information. > > 2. Saving bandwidth. > > > Well, when we've had similar discussions in t

Re: ap_core_output_filter returns APR_SUCCESS on error

2006-08-25 Thread Ruediger Pluem
On 08/25/2006 11:08 PM, Joachim Zobel wrote: > Hi. > > The core output filter is doing the following if the client unexpectedly > closes the connection: > > if (rv != APR_SUCCESS) { >ap_log_cerror(APLOG_MARK, APLOG_INFO, rv, c, > "core_output_filter: writing data t

Re: [EMAIL PROTECTED] -> ASF?

2006-08-26 Thread Ruediger Pluem
On 08/26/2006 12:38 AM, William A. Rowe, Jr. wrote: > > Now the considerations; some here exclaim "too many lists!" I'd argue > that this unique list serves a distinct purpose and needs to be preserved > to its given purpose. Some would argue "but it's not *ASF Apache* module > discussion"

Re: [Vote] create [EMAIL PROTECTED]

2006-09-01 Thread Ruediger Pluem
On 09/01/2006 09:25 PM, William A. Rowe, Jr. wrote: > Project Committee Members... > > Adopt [EMAIL PROTECTED], seeded from [EMAIL PROTECTED] > current subscribers, for module authors to use for peer developer support? > (API 'users', essentially.) > > [ ] +1 > [ ] +/-0 > [ ] -1 +1 o

Re: svn commit: r440337 - in /httpd/httpd/trunk: ./ include/ modules/arch/netware/ modules/experimental/ modules/generators/ modules/http/ modules/mappers/ modules/proxy/ modules/ssl/ server/ server/m

2006-09-05 Thread Ruediger Pluem
On 09/05/2006 03:08 PM, wrote: > Author: trawick > Date: Tue Sep 5 06:08:15 2006 > New Revision: 440337 > > URL: http://svn.apache.org/viewvc?view=rev&rev=440337 > Log: > Replace ap_get_server_version with ap_get_server_banner() and > ap_get_server_description(). > > High-level summary: > > T

Re: svn commit: r440337 - in /httpd/httpd/trunk: ./ include/ modules/arch/netware/ modules/experimental/ modules/generators/ modules/http/ modules/mappers/ modules/proxy/ modules/ssl/ server/ server/m

2006-09-05 Thread Ruediger Pluem
On 09/05/2006 11:49 PM, William A. Rowe, Jr. wrote: > Brian Akins wrote: > >>Ruediger Pluem wrote: >> >> >>>1. If we stick to >>> >>>AP_DECLARE(const char *) ap_get_server_version(void); >>> >>>and do >>> >>&g

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

2006-09-06 Thread Ruediger Pluem
On 09/06/2006 07:58 AM, wrote: > Author: trawick > Date: Tue Sep 5 22:58:14 2006 > New Revision: 440624 > > URL: http://svn.apache.org/viewvc?view=rev&rev=440624 > Log: > propose a concrete patch for backporting ap_get_server_version() > changes to 2.2.x without breaking existing modules > >

Re: svn commit: r442758 - in /httpd/httpd/trunk/modules/generators: mod_cgi.c mod_cgid.c

2006-09-13 Thread Ruediger Pluem
On 09/13/2006 01:44 AM, wrote: > Author: niq > Date: Tue Sep 12 16:44:12 2006 > New Revision: 442758 > > URL: http://svn.apache.org/viewvc?view=rev&rev=442758 > Log: > PR 31759 (mutated) - reported by Jo Rhett > Don't return apr_status_t error value from input filter chain. > > Modified: >

Re: svn commit: r442758 - in /httpd/httpd/trunk/modules/generators: mod_cgi.c mod_cgid.c

2006-09-13 Thread Ruediger Pluem
On 09/13/2006 10:17 PM, Nick Kew wrote: > On Wednesday 13 September 2006 20:33, Ruediger Pluem wrote: > > >>Wouldn't it make sense to return OK even if rv != APR_SUCCESS in the case >>that c->aborted is set, just like in the default handler? > > > I&#x

Re: load balancer and http(s) sticky sessions

2006-09-14 Thread Ruediger Pluem
On 09/14/2006 05:28 PM, Jim Jagielski wrote: > There is a lot of confusion where the users think that > simply adding the stickysession param to the http worker > attribute *adds* the required sticky session info (cookie). > > I'm looking into adding functionality that actually > does that and t

Re: load balancer and http(s) sticky sessions

2006-09-14 Thread Ruediger Pluem
On 09/14/2006 06:14 PM, Jim Jagielski wrote: > > > That's what I'm thinking, sort of like an 'autostickysession' attribute. > We could even have it default to cookies but add something like > 'autostickysession=url' to force URL rewriting and adding a tag > to the end of the URL (for sites that

Re: load balancer and http(s) sticky sessions

2006-09-14 Thread Ruediger Pluem
On 09/14/2006 09:50 PM, Ruediger Pluem wrote: > > > or even > > Header add Set-Cookie "MYCOOKIE=SOMEVALUE.%{BALANCER_WORKER_ROUTE}e; path=/;" > env=BALANCER_ROUTE_CHANGED > ProxyPass /test balancer://mycluster/test stickysession=MYCOOKIE nofailover=On Ok I

Re: load balancer and http(s) sticky sessions

2006-09-15 Thread Ruediger Pluem
On 09/15/2006 07:41 PM, Brian Rectanus wrote: > > I am thinking that there is still not a way to check if > BALANCER_WORKER_ROUTE == BALANCER_SESSION_ROUTE at response time since > I think 'SetEnvIf response ...' will not resolve env vars it did not Yes, this is true. It was also my first tho

Re: mod_cache responsibilities vs mod_xxx_cache provider responsibilities

2006-09-20 Thread Ruediger Pluem
On 09/20/2006 08:27 PM, Issac Goldstand wrote: > > Graham Leggett wrote: > >>On Wed, September 20, 2006 5:27 pm, Brian Akins wrote: >> >> >>>unless 0 is a valid content-length, which it can be. Also, what about >>>when we are reading something in without a know C-L, for example from an >>>orig

Re: mod_cache responsibilities vs mod_xxx_cache provider responsibilities

2006-09-20 Thread Ruediger Pluem
On 09/20/2006 09:59 PM, Issac Goldstand wrote: > > Ruediger Pluem wrote: >> >>First of all I guess you mean: BEFORE the CACHE_SAVE filter :-). >>Yes, there is a reason why we cannot do this: This would create a possible >>DoS, because we have to >>suc

Re: creating new lbmethod for mod_proxy_balancer

2006-09-25 Thread Ruediger Pluem
On 25.09.2006 16:32, Garrett Rooney wrote: > On 9/25/06, Jim Jagielski <[EMAIL PROTECTED]> wrote: > >> Actually, I've added the 'busy' struct element which >> could be used for that... The orig intent was to add >> the mod_jk "busyness" LB method, but it would also >> serve as a flag that the memb

Re: creating new lbmethod for mod_proxy_balancer

2006-09-25 Thread Ruediger Pluem
On 25.09.2006 16:43, Garrett Rooney wrote: > On 9/25/06, Ruediger Pluem <[EMAIL PROTECTED]> wrote: > >> On 25.09.2006 16:32, Garrett Rooney wrote: >> > On 9/25/06, Jim Jagielski <[EMAIL PROTECTED]> wrote: >> > >> >> Actually, I've added t

Re: [PATCH] mod_disk_cache working LFS (filecopy)

2006-09-26 Thread Ruediger Pluem
On 09/26/2006 01:00 PM, Joe Orton wrote: > On Tue, Sep 26, 2006 at 10:52:18AM +0200, Niklas Edmundsson wrote: > >>This patch depends on "mod_disk_cache LFS-aware config" submitted >>earlier and is for trunk. >> >>It makes caching of large files possible on 32bit machines by: >> >>* Realising th

Re: svn commit: r450042 - in /httpd/httpd/trunk: CHANGES modules/cache/mod_disk_cache.c modules/cache/mod_disk_cache.h

2006-09-26 Thread Ruediger Pluem
On 09/26/2006 03:29 PM, wrote: > Author: minfrin > Date: Tue Sep 26 06:29:09 2006 > New Revision: 450042 > > URL: http://svn.apache.org/viewvc?view=rev&rev=450042 > Log: > mod_disk_cache: Make sure that only positive integers are accepted > for the CacheMaxFileSize and CacheMinFileSize parameter

Re: svn commit: r450105 - in /httpd/httpd/trunk: CHANGES modules/cache/mod_disk_cache.c modules/cache/mod_disk_cache.h

2006-09-26 Thread Ruediger Pluem
On 09/26/2006 06:26 PM, wrote: > Author: minfrin > Date: Tue Sep 26 09:26:56 2006 > New Revision: 450105 > > Modified: httpd/httpd/trunk/modules/cache/mod_disk_cache.c > URL: > http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/cache/mod_disk_cache.c?view=diff&rev=450105&r1=450104&r2=450105

Re: svn commit: r450188 - /httpd/httpd/trunk/modules/cache/mod_disk_cache.c

2006-09-26 Thread Ruediger Pluem
On 09/26/2006 10:35 PM, wrote: > Author: minfrin > Date: Tue Sep 26 13:35:42 2006 > New Revision: 450188 > > > Modified: httpd/httpd/trunk/modules/cache/mod_disk_cache.c > URL: > http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/cache/mod_disk_cache.c?view=diff&rev=450188&r1=450187&r2=45

Re: svn commit: r450188 - /httpd/httpd/trunk/modules/cache/mod_disk_cache.c

2006-09-27 Thread Ruediger Pluem
On 09/27/2006 12:22 AM, Graham Leggett wrote: > Ruediger Pluem wrote: > >> Are we sure that we do not iterate too often (> 100) over this during >> the lifetime >> of a request? I would say 'No, we do not iterate too often', but I >> think a c

Re: mod_proxy and balancer problems

2006-09-29 Thread Ruediger Pluem
On 09/29/2006 09:53 AM, Mathias Herberts wrote: > > So redirection is only used when handling session bound requests which > is exactly the opposite of what the documentation said. > > The goal of graceful worker shutdown is therefore not achieved as > there is no way to have session bound req

Re: svn commit: r451006 - in /httpd/httpd/trunk/modules/generators: mod_cgi.c mod_cgid.c

2006-09-29 Thread Ruediger Pluem
On 09/29/2006 03:49 PM, Garrett Rooney wrote: > On 9/29/06, Joe Orton <[EMAIL PROTECTED]> wrote: > >> On Thu, Sep 28, 2006 at 08:15:44PM -, [EMAIL PROTECTED] wrote: >> > --- httpd/httpd/trunk/modules/generators/mod_cgi.c (original) >> > +++ httpd/httpd/trunk/modules/generators/mod_cgi.c Thu

Re: mod_proxy and balancer problems

2006-09-29 Thread Ruediger Pluem
On 09/29/2006 06:28 PM, Joost de Heer wrote: >> I tend to say that >> >> disabled means: Take requests if their routing information matches >> mine but >> no other ones. >> stopped means: Take no requests at all. >> enabled means: Take all requests. > > > Wouldn't 'standby' be a

Re: svn commit: r451575 - in /httpd/httpd/trunk/modules/proxy: mod_proxy.c mod_proxy.h proxy_util.c

2006-09-30 Thread Ruediger Pluem
On 09/30/2006 12:41 PM, wrote: > Author: mturk > Date: Sat Sep 30 03:41:51 2006 > New Revision: 451575 > > URL: http://svn.apache.org/viewvc?view=rev&rev=451575 > Log: > Enable retry=0 for the worker. > This allows to have an option to always retry > the workers in error state instead using a >

Re: svn commit: r451582 - /httpd/httpd/trunk/modules/proxy/mod_proxy.h

2006-10-02 Thread Ruediger Pluem
On 09/30/2006 01:26 PM, [EMAIL PROTECTED] wrote: > Author: mturk > Date: Sat Sep 30 04:26:26 2006 > New Revision: 451582 > > URL: http://svn.apache.org/viewvc?view=rev&rev=451582 > Log: > Move new struct member to the end of the struct, > so we can keep the compatibility. Sorry for nitpicking,

Re: Coding style

2006-10-02 Thread Ruediger Pluem
On 10/02/2006 08:59 PM, Garrett Rooney wrote: > On 10/2/06, Nick Kew <[EMAIL PROTECTED]> wrote: > >> We have a bunch of new bug reports[1], detailing bugs of the form >> >>if ((rv = do_something(args) == APR_SUCCESS)) >> >> for >> >> if ((rv = do_something(args)) == APR_SUCCESS) >> >> Of

Re: svn commit: r451580 - /httpd/httpd/trunk/modules/proxy/proxy_util.c

2006-10-02 Thread Ruediger Pluem
On 09/30/2006 01:04 PM, wrote: > Author: mturk > Date: Sat Sep 30 04:04:46 2006 > New Revision: 451580 > > URL: http://svn.apache.org/viewvc?view=rev&rev=451580 > Log: > Add alternative is_socket_connected implementation. > It works on win32 and linux for sure, so that's why > I put it inside th

Re: Coding style

2006-10-02 Thread Ruediger Pluem
On 10/02/2006 11:15 PM, Joachim Zobel wrote: > Am Montag, den 02.10.2006, 19:56 +0100 schrieb Nick Kew: > >>Can I suggest adopting this as a guideline for new code, >>to avoid this kind of bug? > > > Is there an apache coding style guide? Yes of course: http://httpd.apache.org/dev/styleguide

Re: Coding style

2006-10-02 Thread Ruediger Pluem
On 10/02/2006 11:12 PM, Nick Kew wrote: > On Monday 02 October 2006 21:39, Ruediger Pluem wrote: > > >>In general I agree with the above, but there are situations were the old >>style really makes sense, e.g (from log_child in server/log.c): >> >>if (((rc =

Re: svn commit: r451580 - /httpd/httpd/trunk/modules/proxy/proxy_util.c

2006-10-05 Thread Ruediger Pluem
On 05.10.2006 14:08, Jim Jagielski wrote: > > On Oct 2, 2006, at 4:45 PM, Ruediger Pluem wrote: > >> >> >> On 09/30/2006 01:04 PM, wrote: >> >>> Author: mturk >>> Date: Sat Sep 30 04:04:46 2006 >>> New Revision: 451580 >>&

Re: mod_proxy and balancer problems

2006-10-07 Thread Ruediger Pluem
On 10/05/2006 11:07 AM, Mathias Herberts wrote: > As I mentioned a few days ago here are my patches for mod_proxy. > > They modify mod_proxy behaviour in several ways to better fit our > production environment where we heavily use Apache and Tomcat. > > Just so you understand our context, we us

Re: svn commit: r454683 - in /httpd/httpd/trunk: CHANGES docs/manual/mod/mod_rewrite.xml modules/mappers/mod_rewrite.c

2006-10-10 Thread Ruediger Pluem
On 10/10/2006 12:33 PM, wrote: > Author: niq > Date: Tue Oct 10 03:33:06 2006 > New Revision: 454683 > > URL: http://svn.apache.org/viewvc?view=rev&rev=454683 > Log: > Add SQL Query capability to RewriteMap > > Modified: httpd/httpd/trunk/modules/mappers/mod_rewrite.c > URL: > http://svn.apa

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

2006-10-10 Thread Ruediger Pluem
On 10/10/2006 11:58 PM, wrote: > Author: minfrin > Date: Tue Oct 10 14:58:29 2006 > New Revision: 462594 > > URL: http://svn.apache.org/viewvc?view=rev&rev=462594 > Log: > Propose some backports. > > Modified: > httpd/httpd/branches/2.2.x/STATUS > > Modified: httpd/httpd/branches/2.2.x/STA

Re: svn commit: r462696 - in /httpd/httpd/trunk: CHANGES modules/cache/mod_disk_cache.c modules/cache/mod_disk_cache.h

2006-10-11 Thread Ruediger Pluem
On 10/11/2006 06:31 AM, wrote: > Author: minfrin > Date: Tue Oct 10 21:31:18 2006 > New Revision: 462696 > > URL: http://svn.apache.org/viewvc?view=rev&rev=462696 > Log: > mod_disk_cache: Implement read-while-caching. > > Modified: > httpd/httpd/trunk/CHANGES > httpd/httpd/trunk/module

Issue with persistent http proxy backend connection

2006-10-11 Thread Ruediger Pluem
There is an issue with the proxy code that if a request is sent over a persistent backend connection (currently only looking at the http case, not sure if the same thing can happen for other backends like ajp and fastcgi) it could happen that this connection gets closed by the backend for timeout

Re: Issue with persistent http proxy backend connection

2006-10-12 Thread Ruediger Pluem
On 10/11/2006 11:09 PM, Jeff Trawick wrote: > On 10/11/06, Ruediger Pluem <[EMAIL PROTECTED]> wrote: >> 2. Although GET is mentioned to be idempotent in RFC 2616 (9.1.2) >> along with some other methods >>it is not a good idea to regard a GET / HEAD with query par

Re: Issue with persistent http proxy backend connection

2006-10-12 Thread Ruediger Pluem
On 10/11/2006 11:09 PM, Jeff Trawick wrote: > On 10/11/06, Ruediger Pluem <[EMAIL PROTECTED]> wrote: > >> 3. Sometimes servers (including httpd) include a keep-alive header in >> their response with the parameter >>timeout set. This can give a hint w

Re: Issue with persistent http proxy backend connection

2006-10-12 Thread Ruediger Pluem
On 12.10.2006 13:26, Mladen Turk wrote: > Ruediger Pluem wrote: > >> >> But yes, in theory their remains a race here if the proxy reuses the >> connection >> faster then the backend needs time to close the socket after sending >> the response. >> >

[Fwd: Re: svn commit: r466865 - in /httpd/httpd/trunk: CHANGES docs/manual/mod/mod_authn_dbd.xml modules/aaa/mod_auth.h modules/aaa/mod_authn_dbd.c modules/aaa/mod_authnz_ldap.c]

2006-10-28 Thread Ruediger Pluem
On 10/22/2006 09:11 PM, wrote: > Author: minfrin > Date: Sun Oct 22 12:11:51 2006 > New Revision: 466865 > > URL: http://svn.apache.org/viewvc?view=rev&rev=466865 > Log: > mod_authn_dbd: Export any additional columns queried in the SQL select > into the environment with the name AUTHENTICATE_. Th

[Fwd: Re: svn commit: r467655 - in /httpd/httpd/trunk: CHANGES docs/manual/mod/mod_cache.xml modules/cache/mod_cache.c modules/cache/mod_cache.h]

2006-10-28 Thread Ruediger Pluem
On 10/27/2006 09:41 PM, William A. Rowe, Jr. wrote: > Graham Leggett wrote: > >>I see lots of comments on the code, but the comments are summarised as >>"the cache is fine as it is". It isn't. If it was fine, key users of >>network caching wouldn't be standing up saying they're using something >>

[Fwd: Re: Apache 2.2.3 mod_proxy issue]

2006-10-28 Thread Ruediger Pluem
On 10/27/2006 06:20 PM, Jess Holle wrote: > > On the other hand, if I use: > >ProxyPass /jsp-examples ajp://localhost:8010/jsp-examples > > This works fine! > > I assume I should file a bug against mod_proxy -- or is this a known issue? At least it is known to me :-). See http://iss

[Fwd: Re: Input Filter reading Body]

2006-10-28 Thread Ruediger Pluem
On 10/20/2006 05:03 PM, Christian Verdelli wrote: > > static int redirect_wmlInFilter (ap_filter_t* f, apr_bucket_brigade* > brigade, > ap_input_mode_t mode, apr_read_type_e block, apr_off_t readbytes) { > > apr_bucket* bucket ; > apr_status_t ret ; > > if ( ret = ap_get_brigade(f->next, briga

[Fwd: Re: apr_brigade_create() produces a corrupt brigade]

2006-10-28 Thread Ruediger Pluem
On 10/26/2006 09:48 PM, Graham Leggett wrote: > Hi all, > > I have spent a few hours trying to get to the bottom of this, and and > truly stuck. > > When creating a new brigade inside a filter, I do this: > > r = f->r; > dobj->tmpbb = apr_brigade_create(r->pool, r->connection->bucket_alloc); >

[Fwd: [Fwd: Re: apr_brigade_create() produces a corrupt brigade]]

2006-10-28 Thread Ruediger Pluem
Oops, wrong list. Original Message Subject: [Fwd: Re: apr_brigade_create() produces a corrupt brigade] Date: Sat, 28 Oct 2006 12:02:57 +0200 From: Ruediger Pluem <[EMAIL PROTECTED]> To: dev@httpd.apache.org On 10/26/2006 09:48 PM, Graham Leggett wrote: > Hi all, &g

Re: [Fwd: Re: svn commit: r466865 - in /httpd/httpd/trunk: CHANGES docs/manual/mod/mod_authn_dbd.xml modules/aaa/mod_auth.h modules/aaa/mod_authn_dbd.c modules/aaa/mod_authnz_ldap.c]

2006-10-29 Thread Ruediger Pluem
. On 10/29/2006 01:50 PM, Graham Leggett wrote: > Ruediger Pluem wrote: > >> Does it really make sense to put this in the same environment namespace? >> What if we have rows with the same name here and for the password query? >> Shouldn't the prefix be AUTHN_PREFIX +

Re: [Fwd: Re: svn commit: r467655 - in /httpd/httpd/trunk: CHANGES docs/manual/mod/mod_cache.xml modules/cache/mod_cache.c modules/cache/mod_cache.h]

2006-10-29 Thread Ruediger Pluem
On 10/29/2006 01:56 PM, Graham Leggett wrote: > Ruediger Pluem wrote: > >> Apart from this, Paul created a branch a while ago for mod_cache >> refactoring. >> As it has turned out the whole thing creates some bigger discussion >> and patches >> go in and out

Re: [Fwd: Re: apr_brigade_create() produces a corrupt brigade]

2006-10-29 Thread Ruediger Pluem
On 10/29/2006 01:59 PM, Graham Leggett wrote: > Ruediger Pluem wrote: > >>> This runs fine - a brigade is created, containing a single bucket. >>> >>> The trouble is, the bucket inside the brigade is corrupt - it's name >>> consists of random by

<    1   2   3   4   5   6   7   8   9   10   >