Re: cvs commit: httpd-test/perl-framework/t/protocol nntp-like.t
--On Wednesday, September 29, 2004 3:39 PM +0100 Joe Orton [EMAIL PROTECTED] wrote: OK, the difference is in the handling of an empty Content-Length header. The glibc strtoll does not return an error for an empty string, as C99 requires, and so ap_http_filter treats it exactly as Content-Length: 0. I guess the strto* on your platform does return an error for this case: I'd say a 400 is a better error than a 413 for Content-Length:\r\n but 413 is clearly better than 200, so I've fixed ap_http_filter in HEAD. FWIW, I had failures with Darwin and FreeBSD (might have ran it on Solaris too, but can't recall). Yah, expecting 200 was just plainly wrong in this case. I do think 413 is a bit arbitrary, but is clearly more correct than 200. -- justin
Re: cvs commit: httpd-test/perl-framework/t/protocol nntp-like.t
--On Wednesday, September 29, 2004 10:26 AM +0100 Joe Orton [EMAIL PROTECTED] wrote: Yup, the t_cmp arguments were flipped a while back. FWIW, I think whomever flipped the t_cmp arguments but didn't flip the included test cases at the same time needs a stern talking to. I spent over an hour and a half figuring out why the heck httpd was returning a 200 in that case where a 413 was clearly (or at least more) correct: only to find out that the debug output was swapped. Incredibly, incredibly lame. All I wanted to do last night was add some caching tests: instead I had to fix our tests to pass at all. *sigh* -- justin
[STATUS] (flood) Wed Sep 29 23:45:53 EDT 2004
flood STATUS: -*-text-*- Last modified at [$Date: 2003/07/01 20:55:12 $] Release: 1.0: Released July 23, 2002 milestone-03: Tagged January 16, 2002 ASF-transfer: Released July 17, 2001 milestone-02: Tagged August 13, 2001 milestone-01: Tagged July 11, 2001 (tag lost during transfer) RELEASE SHOWSTOPPERS: * Everything needs to work perfectly Other bugs that need fixing: * I get a SIGBUS on Darwin with our examples/round-robin-ssl.xml config, on the second URL. I'm using OpenSSL 0.9.6c 21 dec 2001. * iPlanet sends Content-length - there is a hack in there now to recognize it. However, all HTTP headers need to be normalized before checking their values. This isn't easy to do. Grr. * OpenSSL 0.9.6 Segfaults under high load. Upgrade to OpenSSL 0.9.6b. Aaron says: I just found a big bug that might have been causing this all along (we weren't closing ssl sockets). How can I reproduce the problem you were seeing to verify if this was the fix? * SEGVs when /tmp/.rnd doesn't exist are bad. Make it configurable and at least bomb with a good error message. (See Doug's patch.) Status: This is fixed, no? * If APR has disabled threads, flood should as well. We might want to have an enable/disable parameter that does this also, providing an error if threads are desired but not available. * flood needs to clear pools more often. With a long running test it can chew up memory very quickly. We should just bite the bullet and create/destroy/clear pools for each level of our model: farm, farmer, profile, url/request-cycle, etc. * APR needs to have a unified interface for ephemeral port exhaustion, but aparently Solaris and Linux return different errors at the moment. Fix this in APR then take advantage of it in flood. * The examples/analyze-relative scripts fail when there are less than 5 unique URLs. Other features that need writing: * More analysis and graphing scripts are needed * Write robust tool (using tethereal perhaps) to take network dumps and convert them to flood's XML format. Status: Justin volunteers. Aaron had a script somewhere that is a start. Jacek is working on a Mozilla application, codename Flood URL bag (much like Live HTTP Headers) and small HTTP proxy. * Get chunked encoding support working. Status: Justin volunteers. He got sidetracked by the httpd implementation of input filtering and never finished this. This is a stopgap until apr-serf is completed. * Maybe we should make randfile and capath runtime directives that come out of the XML, instead of autoconf parameters. * We are using apr_os_thread_current() and getpid() in some places when what we really want is a GUID. The GUID will be used to correlate raw output data with each farmer. We may wish to print a unique ID for each of farm, farmer, profile, and url to help in postprocessing. * We are using strtol() in some places and strtoll() in others. Pick one (Aaron says strtol(), but he's not sure). * Validation of responses (known C-L, specific strings in response) Status: Justin volunteers * HTTP error codes (ie. teach it about 302s) Justin says: Yeah, this won't be with round_robin as implemented. Need a linked list-based profile where we can insert new URLs into the sequence. * Farmer (Single-thread, multiple profiles) Status: Aaron says: If you have threads, then any Farmer can be run as part of any Farm. If you don't have threads, you can currently only run one Farmer named Joe right now (this will be changed so that if you don't have threads, flood will attempt to run all Farmers in serial under one process). * Collective (Single-host, multiple farms) This is a number of Farms that have been fork()ed into child processes. * Megaconglomerate (Multiple hosts each running a collective) This is a number of Collectives running on a number of hosts, invoked via RSH/SSH or maybe even some proprietary mechanism. * Other types of urllists a) Random / Random-weighted b) Sequenced (useful with cookie propogation) c) Round-robin d) Chaining of the above strategies Status: Round-robin is complete. * Other types of reports Status: Aaron says: simple reports are functional. Justin added a new type that simply prints the approx. timestamp when the test was run, and the result as OK/FAIL; it is called easy reports (see flood_easy_reports.h). Furthermore,
[STATUS] (perl-framework) Wed Sep 29 23:45:55 EDT 2004
httpd-test/perl-framework STATUS: -*-text-*- Last modified at [$Date: 2002/03/09 05:22:48 $] Stuff to do: * finish the t/TEST exit code issue (ORed with 0x2C if framework failed) * change existing tests that frob the DocumentRoot (e.g., t/modules/access.t) to *not* do that; instead, have Makefile.PL prepare appropriate subdirectory configs for them. Why? So t/TEST can be used to test a remote server. * problems with -d perl mode, doesn't work as documented Message-ID: [EMAIL PROTECTED] Date: Sat, 20 Oct 2001 12:58:33 +0800 Subject: Re: perldb Tests to be written: * t/apache - simulations of network failures (incomplete POST bodies, chunked and unchunked; missing POST bodies; slooow client connexions, such as taking 1 minute to send 1KiB; ...) * t/modules/autoindex - something seems possibly broken with inheritance on 2.0 * t/ssl - SSLPassPhraseDialog exec: - SSLRandomSeed exec:
Re: cvs commit: httpd-test/perl-framework/t/protocol nntp-like.t
Justin Erenkrantz wrote: --On Wednesday, September 29, 2004 10:26 AM +0100 Joe Orton [EMAIL PROTECTED] wrote: Yup, the t_cmp arguments were flipped a while back. FWIW, I think whomever flipped the t_cmp arguments but didn't flip the included test cases at the same time needs a stern talking to. I spent over an hour and a half figuring out why the heck httpd was returning a 200 in that case where a 413 was clearly (or at least more) correct: only to find out that the debug output was swapped. Incredibly, incredibly lame. yeah, well, that was me. it's difficult to find the time to do everything that needs doing. in this case, the order was swapped to be consistent with other (more popular) Perl testing libraries, but there just weren't enough tuits lying around to make all the changes throughout the perl-framework. the argument at the time was that this was OK(tm) because the only thing affected was the debugging output, not the actual comparison. I'll take the blame for that brain fart too, but again a severe lack of free time got in the way of doing things a bit better. however, those of us that are reasonably active here were aware of this, uh, issue and were changing test files as we touched them for other reasons. so yeah, it sucks, continues to suck, and I'm sorry. I'll buy you a beer or two at apachecon to make up for it :) --Geoff
Re: cvs commit: httpd-test/perl-framework/t/ssl basicauth.t http.t proxy.t
--On Thursday, September 30, 2004 2:54 PM + [EMAIL PROTECTED] wrote: geoff 2004/09/30 07:54:40 Modified:perl-framework/t/apache acceptpathinfo.t chunkinput.t errordoc.t getfile.t limits.t options.t perl-framework/t/filter input_body.t perl-framework/t/http11 chunked.t perl-framework/t/modules alias.t asis.t autoindex2.t cgi.t negotiation.t vhost_alias.t perl-framework/t/php arg.t func5.t getenv.t getlastmod.t ifmodsince.t umask.t var1.t var2.t var3.t perl-framework/t/protocol echo.t perl-framework/t/ssl basicauth.t http.t proxy.t Log: swap t_cmp() arguments where they appeared to not match the current function order. /me adds Geoff to beer list at AC'04. ;-) Yay. Thank you! -- justin
