Re: Getting all supported protocols

2015-08-27 Thread Edward Lu
> Am 26.08.2015 um 18:39 schrieb Edward Lu : > > > > Yes, that's right - the toolkit does itself decide which protocol is > used. However, it decides based on the order of the protocols we pass to > it; that is, it will find the first protocol in the list supported by the

Re: Getting all supported protocols

2015-08-26 Thread Edward Lu
an that the directive ProtocolsHonorOrder has no > longer an effect. Am I right? > > I do not think that is what we want. > > //Stefan > > > Am 26.08.2015 um 17:02 schrieb Edward Lu : > > > > I was experimenting with the new support for declaring protocols for

Getting all supported protocols

2015-08-26 Thread Edward Lu
I was experimenting with the new support for declaring protocols for e.g. ALPN, but with an SSL toolkit other than openssl. This one wants us to pass the entire list of all the protocols the server supports in advance; later, we can request the one protocol that the toolkit negotiated. It looks li

Re: svn commit: r1693140 - in /httpd/test/framework/trunk: c-modules/test_session/ t/htdocs/modules/session/ t/modules/

2015-07-29 Thread Edward Lu
Whoops, sorry, fixed. On Tue, Jul 28, 2015 at 2:59 PM, Eric Covener wrote: > On Tue, Jul 28, 2015 at 2:52 PM, wrote: > > > > Added: > > httpd/test/framework/trunk/c-modules/test_session/ > > httpd/test/framework/trunk/c-modules/test_session/Makefile (with > props) > > > httpd/test/f

Re: Running the test suite on Windows

2015-01-22 Thread Edward Lu
references to it around the web, but all the links were dead. On Wed, Jan 21, 2015 at 1:02 PM, wrote: > - Original Message - > Subject: Running the test suite on Windows > From: "Edward Lu" > Date: 1/21/15 10:05 am > To: dev@httpd.apache.org > > It ap

Running the test suite on Windows

2015-01-21 Thread Edward Lu
It appears that the test framework has a bunch of dependencies on unix, e.g. it runs scripts with "#!/usr/bin/...". Running it under cygwin would fix some of those issues, but unfortunately, running under cygwin perl generates unix-like paths in the conf files which a windows-built httpd can't deal

100% CPU on Win32 processing chunked encoding (trunk)

2015-01-20 Thread Edward Lu
I recently compiled Apache trunk using the CMake build system (which is excellent, by the way), with the intention of trying to fix up the HTTPD test framework to work on Windows. One of the tests seems to have caught something rather severe, with HTTPD threads hanging and taking 100% CPU. It happe

Re: Building APR using cmake on Windows

2014-11-06 Thread Edward Lu
Thanks Jeff. Will move further discussion to the bug/proper mailing list. On Thu, Nov 6, 2014 at 10:15 AM, Jeff Trawick wrote: > On Thu, Oct 30, 2014 at 1:53 PM, Edward Lu wrote: > >> Using Visual Studio 2013 command line tools, on Windows Server 2012 rc2. >> Running eve

Re: Building APR using cmake on Windows

2014-11-04 Thread Edward Lu
On Thu, Oct 30, 2014 at 2:09 PM, Jeff Trawick wrote: > > I guess this is actually the right tarball: > > > https://sites.google.com/a/unimrcp.org/unimrcp/dependencies/apr-1.5.1-patches.zip?attredirects=0&d=1 > Right, sorry. > > See > http://mail-archives.apache.org/mod_mbox/apr-dev/201404.mbox

Building APR using cmake on Windows

2014-10-30 Thread Edward Lu
Using Visual Studio 2013 command line tools, on Windows Server 2012 rc2. Running everything from batch files through Cygwin. I cloned the APR repo from here: https://github.com/apache/apr I downloaded some prebuilt libxml2 and iconv binaries for Windows. Then, I ran the commands: "C:\Program File

Re: Question about async mod_proxy_wstunnel and threads

2014-07-25 Thread Edward Lu
pfd); push_timer2worker(te); } It shouldn't be necessary either, since out_pfd is included in baton->pfds; it gets removed in the for loop just before. On Fri, Jul 25, 2014 at 1:31 PM, Edward Lu wrote: > Re: Patch; yes, I think that works as well, but I'm still not sure it >

Re: Question about async mod_proxy_wstunnel and threads

2014-07-25 Thread Edward Lu
t_pfd); > push_timer2worker(te); > } > -apr_pollset_remove(event_pollset, out_pfd); > +else { > +apr_pollset_remove(event_pollset, out_pfd); > +} > } > out_pfd++; >

Re: Question about async mod_proxy_wstunnel and threads

2014-07-25 Thread Edward Lu
set_remove(); all sockets are removed from the pollset when one of them is signalled. If another one was signalled in the same poll, it will enter this block and simply do nothing (because of the baton->signaled check). I gave it a quick test and it looked like the problem was solved. On Tue, Jul

Re: Question about async mod_proxy_wstunnel and threads

2014-07-22 Thread Edward Lu
I got it to break with these python scripts, as well as with my own test client and server (browser javascript and node.js). Strange though, I can't reproduce it with only 2 clients on either of those programs; I have to get 3 clients connected at once. Past that, I see exactly the behavior you des

Re: stop copying footers to r->headers_in?

2014-06-25 Thread Edward Lu
Patch for trunk as well On Wed, Jun 25, 2014 at 1:20 PM, Edward Lu wrote: > Wanted to follow up on this thread again; here's my latest patch on 2.2.x > that takes some ideas from Joe's patch. I also merge the trailers into the > headers after reading them now, instead of

Re: stop copying footers to r->headers_in?

2014-06-25 Thread Edward Lu
Wanted to follow up on this thread again; here's my latest patch on 2.2.x that takes some ideas from Joe's patch. I also merge the trailers into the headers after reading them now, instead of directly appending them into the headers. I will leave the PROXYREQ_RESPONSE case to someone else, as I st

Re: stop copying footers to r->headers_in?

2014-05-16 Thread Edward Lu
Added mod_log_config changes; I used "O" and "I" for the format letters. Not sure if there are letters that might be more preferred. On Fri, May 9, 2014 at 11:01 AM, Edward Lu wrote: > Here's a 2.2.x backport of Yann's patch with a directive, MergeTrailers

Re: stop copying footers to r->headers_in?

2014-05-12 Thread Edward Lu
Both the things you caught were probably just errors I made in manually merging the patch into 2.2.x; here's a revised version, just in case. Thanks for the review. I'm not really clear on the error-notes discussion. It makes sense that we wouldn't care too much if we don't save the ones from read

Re: stop copying footers to r->headers_in?

2014-05-10 Thread Edward Lu
Here's a 2.2.x backport of Yann's patch with a directive, MergeTrailers, added in to opt-in to the old behavior. By default, it doesn't merge the trailers. I'm working on adding the capability to mod_log_config to log the trailers, but I figured I'd post this patch first to get some review. - Ed