Asst'd Win32 failures [that should simply be skipped]

2002-06-22 Thread William A. Rowe, Jr.
The following tests are all sh scripts that won't work on win32, we should
exclude on non-sh supporting platforms;
apache\acceptpathinfo.t   36   10  27.78%  9-10, 21-24, 33-36
The following test looks like we didn't follow our have apxs test results
(we can't compile modules on win32 right at this instant) before we try
an echo_post_chunk request;
apache\chunkinput.t92  22.22%  2-3
This looks like the same problem again, we never built echo_post before
attempting to run the contentlength.t tests;
apache\contentlength.t203  15.00%  2, 6, 10
I'm entirely unclear what is happening on these two tests, however;
apache\limits.t   102  20.00%  9-10
# Testing LimitRequestBody; should fail
# Chunked transfer-encoding enabled
# testing : Test #9
# expected: 413
# received: 500
not ok 9
# Failed test 9 in apache\limits.t at line 112 fail #2
# Server response:
# 500 (Internal Server Error) EOF
# Client-Date: Sat, 22 Jun 2002 20:47:14 GMT
#
#
#
# Testing LimitRequestBody; should fail
# Chunked transfer-encoding disabled
#lwp request:
#GET http://localhost:8529/apache/limits/ HTTP/1.0
#User-Agent: libwww-perl/5.65
#Content-Type: text/plain
#X-Subtest: 10
#
#aaa[...sparing 
you lots of 'a's in the lwp=2 output...
a
#server response:
#500 (Internal Server Error) EOF instead of reponse status line
#Content-Length: 0
#Client-Date: Sat, 22 Jun 2002 20:47:14 GMT
#X-Content-Length-Note: added by Apache::TestRequest
#
#
# testing : Test #10
# expected: 413
# received: 500
not ok 10
# Failed test 10 in apache\limits.t at line 123 fail #2
# Server response:
# 500 (Internal Server Error) EOF instead of reponse status line
# Content-Length: 0
# Client-Date: Sat, 22 Jun 2002 20:47:14 GMT
# X-Content-Length-Note: added by Apache::TestRequest

This one is just plain silly, looks like a semantics error...
modules\include.t 516  11.76%  46-51
ok 36
ok 37 # Skipping XBitHack tests on this platform
ok 38 # Skipping XBitHack tests on this platform
ok 39 # Skipping XBitHack tests on this platform
ok 40 # Skipping XBitHack tests on this platform
ok 41 # Skipping XBitHack tests on this platform
ok 42 # Skipping XBitHack tests on this platform
ok 43 # Skipping XBitHack tests on this platform
ok 44 # Skipping XBitHack tests on this platform
ok 45 # Skipping XBitHack tests on this platform
FAILED tests 46-51
Failed 6/51 tests, 88.24% okay (-9 skipped tests: 36 okay, 70.59%)
Since when is a skip == failure :-?
I won't bother reporting our failures in ssl\*.t for the moment till I get
a handle on what ssl perl module errors are going on.
So if someone who groks this all would add [and fix the existing borked]
'skipping' statuses to these tests, or share some observation about what
is up with the limits.t test, I would be most appreciative.
Bill


Re: Asst'd Win32 failures [that should simply be skipped]

2002-06-22 Thread Randy Kobes
On Sat, 22 Jun 2002, William A. Rowe, Jr. wrote:

 This one is just plain silly, looks like a semantics error...

 modules\include.t 516  11.76%  46-51

 ok 36
 ok 37 # Skipping XBitHack tests on this platform
 ok 38 # Skipping XBitHack tests on this platform
 ok 39 # Skipping XBitHack tests on this platform
 ok 40 # Skipping XBitHack tests on this platform
 ok 41 # Skipping XBitHack tests on this platform
 ok 42 # Skipping XBitHack tests on this platform
 ok 43 # Skipping XBitHack tests on this platform
 ok 44 # Skipping XBitHack tests on this platform
 ok 45 # Skipping XBitHack tests on this platform
 FAILED tests 46-51
   Failed 6/51 tests, 88.24% okay (-9 skipped tests: 36 okay, 70.59%)

 Since when is a skip == failure :-?

The following

===
Index: t/modules/include.t
===
RCS file: /home/cvspublic/httpd-test/perl-framework/t/modules/include.t,v
retrieving revision 1.21
diff -u -r1.21 include.t
--- t/modules/include.t 20 Jun 2002 04:05:04 -  1.21
+++ t/modules/include.t 22 Jun 2002 21:31:00 -
@@ -141,9 +141,9 @@
 for (1..9) {
 skip Skipping XBitHack tests on this platform, 1;
 }
-exit;
+#exit;
 }
-
+else {
 ### XBITHACK TESTS
 # test xbithack off
 $doc = xbithack/off/test.html;
@@ -189,7 +189,7 @@
  XBitHack full [0554]
 );

-
+}
 ### MOD_BUCKETEER+MOD_INCLUDE TESTS
 # we can use mod_bucketeer to create edge conditions for mod_include, since
 # it allows us to create bucket and brigade boundaries wherever we want

===

is one way of fixing this.

best regards,
randy kobes



Re: daedalus pegged

2002-06-22 Thread Pier Fumagalli

Greg Ames [EMAIL PROTECTED] wrote:

 Cliff Woolley wrote:
 
 Well, I guess our security advisory worked.  Daedalus is totally maxxed
 out at 700 concurrent requests
 
 holy cripe!
 
 (which I assume is the maxclients setting).
 
 yep.
 
 MANY people downloading the new releases of Apache (expecially
 1.3.26.tar.gz and the win32 binary of the same version).
 
 Good to see our efforts taken seriously.
 
 It's back down to 560 now.  We've got a bunch of headroom available on
 MaxClients, but I don't want to bump it up a huge amount at times like this
 without watching it closely.
 
 Thanks for the heads up.

I'm rsyincing now the distribution over onto nagoya... Given it's an
*.apache.org machine, I believe that people might feel more safe to
download it from there... I'm adding a mirror to our set...

One other thing we might want to consider is putting a link to the download
site over onto the httpd site, pointing to /dist, with something like

  A href=httpd://www.apache.org/dist/httpdUpdate NOW!/A

Pier
--
[Perl] combines all the worst aspects of C and Lisp:  a billion of different
sublanguages in  one monolithic executable.  It combines the power of C with
the readability of PostScript. [Jamie Zawinski - DNA Lounge - San Francisco]




Re: cvs commit: httpd-2.0/modules/generators mod_cgi.c mod_cgid.c

2002-06-22 Thread William A. Rowe, Jr.

Looking for both faults (passing the next filter when we are doing lookups,
and passing NULL when we are going to serve the content) ... it really
seems that;

D:\clean\httpd-2.0\modules\mappers\mod_negotiation.c(2550):
 sub_req = ap_sub_req_lookup_file(variant-file_name, r, NULL);

SHOULD be setting the next filter.  Anyone want to dig on this?

Bill

wrowe   2002/06/22 09:32:45

   Modified:include  http_request.h
modules/generators mod_cgi.c mod_cgid.c
   Log:
 Note the changed meaning of the NULL next_filter argument to the
 ap_sub_req_lookup() family, and fix a few oddball cases (those are,
 PATH_TRANSLATED reference issues.)





RE: cvs commit: httpd-2.0/modules/generators mod_cgi.c mod_cgid.c

2002-06-22 Thread Ryan Bloom

 From: William A. Rowe, Jr. [mailto:[EMAIL PROTECTED]]
 Sent: Saturday, June 22, 2002 9:45 AM
 To: [EMAIL PROTECTED]
 Subject: Re: cvs commit: httpd-2.0/modules/generators mod_cgi.c
mod_cgid.c
 
 Looking for both faults (passing the next filter when we are doing
 lookups,
 and passing NULL when we are going to serve the content) ... it really
 seems that;
 
 D:\clean\httpd-2.0\modules\mappers\mod_negotiation.c(2550):
  sub_req = ap_sub_req_lookup_file(variant-file_name, r,
NULL);
 
 SHOULD be setting the next filter.  Anyone want to dig on this?

I am missing something.  The third argument has nothing to do with
whether the request will be served.  It has to do with how the filter
stack is initialized in the sub-request.  If the argument is NULL, the
sub-requests filters are set to the protocol filters of the original
request.  If the argument isn't NULL, then the sub-request's filters are
initialized to the filter passed in.  In either case, the data can
either be sent or not.

Please revert the doc portions of your last change, they are incorrect.
The code is safe to leave, because the requests aren't actually served.
However, mod_negotiation is 100% correct, and should NOT be changed.

Ryan

 
 Bill
 
 wrowe   2002/06/22 09:32:45
 
Modified:include  http_request.h
 modules/generators mod_cgi.c mod_cgid.c
Log:
  Note the changed meaning of the NULL next_filter argument to the
  ap_sub_req_lookup() family, and fix a few oddball cases (those
are,
  PATH_TRANSLATED reference issues.)
 





worker MPM shutdown

2002-06-22 Thread Brian Pane

During the worker MPM non-graceful shutdown, the signal_threads()
function attempts to close all open sockets.

I have two major objections to this:

1) It's not necessarily safe to close a socket that another thread
   is using.  Note that apr_socket_close() calls the pool cleanup
   on the pool from which the socket was allocated--bad news if
   one of the active worker threads happens to be, say, registering
   a new cleanup in the same pool at the same time.

2) It appears to be contributing to the fact that non-graceful
   shutdown doesn't work.  Without the socket shutdown loop,
   the child processes shut down promptly.

As I understand it, the motivation for closing the sockets during
shutdown was to try to fix a race condition in which an active
worker thread might be trying to write to a socket at the same
time that the underlying pool was being destroyed (due to the
recursive destruction of APR's global_pool during apr_terminate()).

If so, I think the right solution is to add a way to create
parentless pools that aren't implicitly added as children to
the global pool, so that a worker thread's pool won't disappear
before that thread does.  Is there any specific reason why we're
not doing this already?

Thanks,
--Brian





BUG: http_vhost.c:fix_hostname

2002-06-22 Thread Perry Harrington

There is a bug in fix_hostname.  The comment above function says that the hostname
is lowercased, but it's not.

the line which reads:

*dst++ = *src++;

in context of:

goto bad;
else
break;
}
*dst++ = *src++;
}
/* strip trailing gubbins */
if (dst  host  dst[-1] == '.') {
dst[-1] = '\0';
} else {

should read:

*dst++ = tolower(*src++);

Thanks,

--Perry

-- 
Perry Harrington Director ofzelur xuniL  ()
perry at webcom dot com  System ArchitectureThink Blue.  /\

Those who would give up essential liberty to purchase a little temporary safety
deserve neither liberty or safety. Nor, are they likely to end up with either.
 -- Benjamin Franklin