svn commit: r1710385 - /serf/trunk/outgoing.c

2015-10-24 Thread rhuijben
Author: rhuijben Date: Sat Oct 24 20:45:35 2015 New Revision: 1710385 URL: http://svn.apache.org/viewvc?rev=1710385=rev Log: * outgoing.c (store_ipaddresses_in_config): Increase buffer space to allow formatting ipv6 addresses and check the result to avoid missing APR_ENOSPC and thereby

svn commit: r1710687 - /serf/trunk/protocols/http2_protocol.c

2015-10-26 Thread rhuijben
Author: rhuijben Date: Mon Oct 26 20:19:07 2015 New Revision: 1710687 URL: http://svn.apache.org/viewvc?rev=1710687=rev Log: Checkpoint the current http2 protocol implementation. This now dumps the completely decoded hpack headers when logging is set. This adds some helper functions to quickly

svn commit: r1710998 - in /serf/trunk/protocols: http2_protocol.c http2_protocol.h

2015-10-28 Thread rhuijben
Author: rhuijben Date: Wed Oct 28 12:11:37 2015 New Revision: 1710998 URL: http://svn.apache.org/viewvc?rev=1710998=rev Log: Improve the http2 protocol handler's event handling by adding a bit more abstraction on writing frames. * protocols/http2_protocol.c (serf_http2_stream_t): Move

svn commit: r1711241 - /serf/trunk/test/MockHTTPinC/MockHTTP_server.c

2015-10-29 Thread rhuijben
Author: rhuijben Date: Thu Oct 29 12:56:14 2015 New Revision: 1711241 URL: http://svn.apache.org/viewvc?rev=1711241=rev Log: * test/MockHTTPinC/MockHTTP_server.c (connectToServer): Remove APR_SO_REUSEADDR. We don't use an explicit connect-from address, that may have required this. Modified

svn commit: r1710902 - /serf/trunk/connection_request.c

2015-10-27 Thread rhuijben
Author: rhuijben Date: Tue Oct 27 23:00:23 2015 New Revision: 1710902 URL: http://svn.apache.org/viewvc?rev=1710902=rev Log: * connection_request.c (serf__provide_credentials): Resolve shadowed variable warning. Modified: serf/trunk/connection_request.c Modified: serf/trunk

svn commit: r1710903 - /serf/trunk/buckets/ssl_buckets.c

2015-10-27 Thread rhuijben
Author: rhuijben Date: Tue Oct 27 23:03:48 2015 New Revision: 1710903 URL: http://svn.apache.org/viewvc?rev=1710903=rev Log: * buckets/ssl_buckets.c (ssl_decrypt): Remove unused variable. (serf_ssl_negotiate_protocol): Change type of output buffer to match openssl. Apply some cast right

svn commit: r1711074 - /serf/trunk/test/serf_get.c

2015-10-28 Thread rhuijben
Author: rhuijben Date: Wed Oct 28 17:28:59 2015 New Revision: 1711074 URL: http://svn.apache.org/viewvc?rev=1711074=rev Log: Add support for h2direct to serf_get. This allows running test requests against servers such as nghttpd and Apache httpd 2.4.17, that may allow speaking http2 directly

svn commit: r1711193 - /serf/trunk/test/serf_get.c

2015-10-29 Thread rhuijben
Author: rhuijben Date: Thu Oct 29 08:43:14 2015 New Revision: 1711193 URL: http://svn.apache.org/viewvc?rev=1711193=rev Log: * test/serf_get.c (print_usage): Use the same algorithm as apr_getopt_long() to determine how many options there are in the list. Modified: serf/trunk/test

svn commit: r1710661 - /serf/trunk/buckets/http2_frame_buckets.c

2015-10-26 Thread rhuijben
Author: rhuijben Date: Mon Oct 26 17:38:29 2015 New Revision: 1710661 URL: http://svn.apache.org/viewvc?rev=1710661=rev Log: Make the http2 frame bucket a bit easier to use by extending the kind of arguments that can be passed to the create function. * buckets/http2_frame_buckets.c

svn commit: r1709919 - /serf/trunk/buckets/ssl_buckets.c

2015-10-21 Thread rhuijben
Author: rhuijben Date: Wed Oct 21 21:07:56 2015 New Revision: 1709919 URL: http://svn.apache.org/viewvc?rev=1709919=rev Log: Following up on r1708829, handle the fallback of protocol negotiation after the handshake has finished instead of only after the entire negotiation has finished

svn propchange: r1709933 - svn:log

2015-10-21 Thread rhuijben
Author: rhuijben Revision: 1709933 Modified property: svn:log Modified: svn:log at Wed Oct 21 23:17:57 2015 -- --- svn:log (original) +++ svn:log Wed Oct 21 23:17:57 2015 @@ -5,7 +5,7 @@ the connected state when

svn commit: r1709934 - /serf/trunk/outgoing.c

2015-10-21 Thread rhuijben
Author: rhuijben Date: Wed Oct 21 23:19:21 2015 New Revision: 1709934 URL: http://svn.apache.org/viewvc?rev=1709934=rev Log: * outgoing.c (serf__process_connection): Following up on r1709933, tweak the code to not only handle the connect event on the first APR_POLLOUT event, but also

svn commit: r1713788 - in /serf/trunk: buckets/buckets.c buckets/dechunk_buckets.c buckets/deflate_buckets.c buckets/hpack_buckets.c buckets/http2_frame_buckets.c buckets/prefix_buckets.c serf_bucket_

2015-11-11 Thread rhuijben
Author: rhuijben Date: Wed Nov 11 09:39:09 2015 New Revision: 1713788 URL: http://svn.apache.org/viewvc?rev=1713788=rev Log: Implement a best effort serf_default_readline() via read() and peek() to be used on buckets that don't have a specialized implementation and currently use a NULL function

svn commit: r1713810 - in /serf/trunk: ./ buckets/

2015-11-11 Thread rhuijben
Author: rhuijben Date: Wed Nov 11 11:12:22 2015 New Revision: 1713810 URL: http://svn.apache.org/viewvc?rev=1713810=rev Log: As we already have buckets v2, add a readline2() api to it to allow requesting reading of a line upto a certain maximum. All users that don't need strict limiting should

svn commit: r1713795 - /serf/trunk/test/test_buckets.c

2015-11-11 Thread rhuijben
Author: rhuijben Date: Wed Nov 11 10:06:06 2015 New Revision: 1713795 URL: http://svn.apache.org/viewvc?rev=1713795=rev Log: Commit some minor correctness fixes to the bucket tests to resolve test issues found when enabling the bucket test wrappers. * test/test_buckets.c (SERF_DEBUG_BUCKET_USE

svn commit: r1713800 - in /serf/trunk: buckets/buckets.c buckets/limit_buckets.c test/test_buckets.c

2015-11-11 Thread rhuijben
Author: rhuijben Date: Wed Nov 11 10:20:53 2015 New Revision: 1713800 URL: http://svn.apache.org/viewvc?rev=1713800=rev Log: Resolve an issue in the limit buckets where readline previously allowed reading further than the set limit. * buckets/buckets.c (serf_default_readline): Fix several

svn commit: r1713783 - /serf/trunk/buckets/request_buckets.c

2015-11-11 Thread rhuijben
Author: rhuijben Date: Wed Nov 11 09:19:06 2015 New Revision: 1713783 URL: http://svn.apache.org/viewvc?rev=1713783=rev Log: Resolve some SERF_DEBUG_BUCKET_USE issues by calling functions directly via the vtable. * buckets/request_buckets.c (serf_request_read, serf_request_readline

svn commit: r1713802 - /serf/trunk/buckets/buckets.c

2015-11-11 Thread rhuijben
Author: rhuijben Date: Wed Nov 11 10:27:55 2015 New Revision: 1713802 URL: http://svn.apache.org/viewvc?rev=1713802=rev Log: * buckets/buckets.c (serf_get_type): Note a possible future optimization. Modified: serf/trunk/buckets/buckets.c Modified: serf/trunk/buckets/buckets.c URL: http

svn commit: r1713822 - in /serf/trunk: buckets/response_body_buckets.c test/mock_sock_buckets.c

2015-11-11 Thread rhuijben
Author: rhuijben Date: Wed Nov 11 12:16:12 2015 New Revision: 1713822 URL: http://svn.apache.org/viewvc?rev=1713822=rev Log: * test/mock_sock_buckets.c (serf_bucket_type_mock_socket): Properly implement bucket v2. Modified: serf/trunk/buckets/response_body_buckets.c serf/trunk/test

svn commit: r1713906 - in /serf/trunk: ./ buckets/ test/

2015-11-11 Thread rhuijben
Author: rhuijben Date: Wed Nov 11 18:09:06 2015 New Revision: 1713906 URL: http://svn.apache.org/viewvc?rev=1713906=rev Log: Revert the bucket type changes of r1713810, r1713817, by switching the few places where we actually use the new features of a limiting readline to calling

svn commit: r1713894 - /serf/trunk/buckets/barrier_buckets.c

2015-11-11 Thread rhuijben
Author: rhuijben Date: Wed Nov 11 17:15:16 2015 New Revision: 1713894 URL: http://svn.apache.org/viewvc?rev=1713894=rev Log: * buckets/barrier_buckets.c (serf_barrier_read_for_sendfile): Resolve endless loop. Found by: gstein Modified: serf/trunk/buckets/barrier_buckets.c Modified: serf

svn commit: r1713861 - /serf/trunk/buckets/deflate_buckets.c

2015-11-11 Thread rhuijben
Author: rhuijben Date: Wed Nov 11 14:46:37 2015 New Revision: 1713861 URL: http://svn.apache.org/viewvc?rev=1713861=rev Log: Following up on r1713852, tweak the deflate2 function to a function that is called before the actual reading starts. * buckets/deflate_buckets.c (deflate_read2): Rename

svn commit: r1713867 - /serf/trunk/buckets/deflate_buckets.c

2015-11-11 Thread rhuijben
Author: rhuijben Date: Wed Nov 11 15:16:07 2015 New Revision: 1713867 URL: http://svn.apache.org/viewvc?rev=1713867=rev Log: Following up on r1713866, implement peek support on the deflate stream. This indirectly makes it safe to use the readline functions on this bucket type. (See issue

svn commit: r1713834 - in /serf/trunk: buckets/buckets.c test/test_buckets.c

2015-11-11 Thread rhuijben
Author: rhuijben Date: Wed Nov 11 13:16:46 2015 New Revision: 1713834 URL: http://svn.apache.org/viewvc?rev=1713834=rev Log: Extend the linebuf crlf test to show that we have a huge problem with some bucket types. I really hope that nobody encountered this in the real world. I have no proper

svn commit: r1713866 - /serf/trunk/buckets/deflate_buckets.c

2015-11-11 Thread rhuijben
Author: rhuijben Date: Wed Nov 11 15:09:00 2015 New Revision: 1713866 URL: http://svn.apache.org/viewvc?rev=1713866=rev Log: Following up on r1713852 and r1713861, tweak the temporary deflate_read3 function to serf_deflate_refill(), extracting the actual reading back to serf_deflate_read

svn commit: r1713817 - in /serf/trunk: buckets/dechunk_buckets.c serf.h test/test_buckets.c

2015-11-11 Thread rhuijben
Author: rhuijben Date: Wed Nov 11 12:01:19 2015 New Revision: 1713817 URL: http://svn.apache.org/viewvc?rev=1713817=rev Log: Make the dechunk bucket properly implement readline and peek via delegating to the underlying stream. * buckets/dechunk_buckets.c (wait_for_chunk): New function

svn commit: r1713829 - /serf/trunk/test/test_buckets.c

2015-11-11 Thread rhuijben
Author: rhuijben Date: Wed Nov 11 12:33:20 2015 New Revision: 1713829 URL: http://svn.apache.org/viewvc?rev=1713829=rev Log: Following up on r1713822, add regression test on the problem fixed on the body bucket. Modified: serf/trunk/test/test_buckets.c Modified: serf/trunk/test

svn commit: r1713837 - /serf/trunk/buckets/limit_buckets.c

2015-11-11 Thread rhuijben
Author: rhuijben Date: Wed Nov 11 13:25:41 2015 New Revision: 1713837 URL: http://svn.apache.org/viewvc?rev=1713837=rev Log: Make the limit stream pass readline requests while still guarding the limit. * buckets/limit_buckets.c (serf_limit_read): Simplify check a bit. (serf_limit_readline2

svn commit: r1713830 - in /serf/trunk: buckets/barrier_buckets.c test/test_buckets.c

2015-11-11 Thread rhuijben
Author: rhuijben Date: Wed Nov 11 12:44:53 2015 New Revision: 1713830 URL: http://svn.apache.org/viewvc?rev=1713830=rev Log: Following up on r1713829, extend the test a bit more to also verify the second response. Implement a few more forwards on the barrier buckets to make this possible

svn commit: r1713852 - /serf/trunk/buckets/deflate_buckets.c

2015-11-11 Thread rhuijben
Author: rhuijben Date: Wed Nov 11 14:36:21 2015 New Revision: 1713852 URL: http://svn.apache.org/viewvc?rev=1713852=rev Log: In preparation for improving the capabilities of the deflate stream, split the serf_deflate_read() function in 3 functions that now call each other. No functional changes

svn commit: r1713511 - in /serf/trunk: outgoing.c ssltunnel.c

2015-11-09 Thread rhuijben
Author: rhuijben Date: Mon Nov 9 19:01:08 2015 New Revision: 1713511 URL: http://svn.apache.org/viewvc?rev=1713511=rev Log: Following up on r1713500, don't make the number of requests and responses go out of sync, but fix the calculation in outgoing.c instead. * outgoing.c

svn commit: r1713603 - /serf/trunk/test/test_context.c

2015-11-10 Thread rhuijben
Author: rhuijben Date: Tue Nov 10 10:17:09 2015 New Revision: 1713603 URL: http://svn.apache.org/viewvc?rev=1713603=rev Log: * test/test_context.c (queue_part2): Really queue the second part. Use aggregate allocator. Found by: -DSERF_DEBUG_BUCKET_USE Modified: serf/trunk/test

svn commit: r1713595 - /serf/trunk/test/test_buckets.c

2015-11-10 Thread rhuijben
Author: rhuijben Date: Tue Nov 10 08:58:15 2015 New Revision: 1713595 URL: http://svn.apache.org/viewvc?rev=1713595=rev Log: Use the bucket allocator in the deflate bucket tests. Allocating things in pools is not testing our real world scenarios and uses pools in +- the worst performance case

svn commit: r1714111 - /serf/trunk/connection_request.c

2015-11-12 Thread rhuijben
Author: rhuijben Date: Thu Nov 12 20:17:23 2015 New Revision: 1714111 URL: http://svn.apache.org/viewvc?rev=1714111=rev Log: In the request cleanup/destroy logic: tweak SERF_DEBUG_BUCKET_USE code to verify checks while the allocator is still in the right state. * connection_request.c

svn commit: r1714110 - /serf/trunk/buckets/hpack_buckets.c

2015-11-12 Thread rhuijben
Author: rhuijben Date: Thu Nov 12 20:14:23 2015 New Revision: 1714110 URL: http://svn.apache.org/viewvc?rev=1714110=rev Log: Simplify the hpack bucket a bit, now that it doesn't need a buffering layer to support readline properly. * buckets/hpack_buckets.c (serf_hpack_decode_ctx_t): Remove

svn propchange: r1699757 - svn:log

2015-11-12 Thread rhuijben
Author: rhuijben Revision: 1699757 Modified property: svn:log Modified: svn:log at Thu Nov 12 09:37:12 2015 -- --- svn:log (original) +++ svn:log Thu Nov 12 09:37:12 2015 @@ -1,4 +1,4 @@ -Implement at runtim configurable

svn commit: r1714116 - /serf/trunk/buckets/buckets.c

2015-11-12 Thread rhuijben
Author: rhuijben Date: Thu Nov 12 20:26:20 2015 New Revision: 1714116 URL: http://svn.apache.org/viewvc?rev=1714116=rev Log: Resolve another case where we don't set the output length when trying to read a line fails with some error. * buckets/buckets.c (serf_databuf_readline): Set length to 0

svn commit: r1714247 - /serf/trunk/buckets/hpack_buckets.c

2015-11-13 Thread rhuijben
Author: rhuijben Date: Fri Nov 13 18:15:58 2015 New Revision: 1714247 URL: http://svn.apache.org/viewvc?rev=1714247=rev Log: * buckets/hpack_buckets.c (serf_hpack_entry_t): Switch to booleans to our new serf private C99 bool. (*): Update usages to use the standardized macros. Modified

svn commit: r1714246 - in /serf/trunk: buckets/aggregate_buckets.c buckets/limit_buckets.c buckets/split_buckets.c serf.h serf_bucket_types.h test/test_buckets.c

2015-11-13 Thread rhuijben
Author: rhuijben Date: Fri Nov 13 18:12:12 2015 New Revision: 1714246 URL: http://svn.apache.org/viewvc?rev=1714246=rev Log: Implement split buckets as a real fix for the data lifetime issues I uncovered in r1713435, by abusing the limit bucket. The previous fix did solve the symptoms

svn commit: r1714299 - /serf/trunk/buckets/http2_frame_buckets.c

2015-11-14 Thread rhuijben
Author: rhuijben Date: Sat Nov 14 10:36:15 2015 New Revision: 1714299 URL: http://svn.apache.org/viewvc?rev=1714299=rev Log: * buckets/http2_frame_buckets.c (serf__bucket_http2_unframe_read_info): Diagnose one specific kind of frame size error as a protocol error. This is most likely

svn commit: r1714297 - in /serf/trunk: buckets/event_buckets.c outgoing.c serf_private.h

2015-11-14 Thread rhuijben
Author: rhuijben Date: Sat Nov 14 09:36:08 2015 New Revision: 1714297 URL: http://svn.apache.org/viewvc?rev=1714297=rev Log: Redefine the event bucket as a wrapping bucket, to remove dependencies on implementation details on the 'aggregation' stream around the event bucket. Calculate total

svn commit: r1714307 - in /serf/trunk: buckets/split_buckets.c protocols/http2_protocol.c protocols/http2_protocol.h protocols/http2_stream.c

2015-11-14 Thread rhuijben
Author: rhuijben Date: Sat Nov 14 12:24:27 2015 New Revision: 1714307 URL: http://svn.apache.org/viewvc?rev=1714307=rev Log: Use the recently added split buckets to allow sending more hpack headers than that fit in a single HEADERS frame. * buckets/split_buckets.c (serf_bucket_split_create

svn commit: r1714457 - in /serf/trunk: SConstruct incoming.c

2015-11-15 Thread rhuijben
Author: rhuijben Date: Sun Nov 15 14:41:20 2015 New Revision: 1714457 URL: http://svn.apache.org/viewvc?rev=1714457=rev Log: Fix some breakage when using different setups. * SConstruct Add new file. * incoming.c (serf_incoming_create2): Move declaration up a bit. Modified: serf/trunk

svn commit: r1714445 - /serf/trunk/config_store.c

2015-11-15 Thread rhuijben
Author: rhuijben Date: Sun Nov 15 12:23:08 2015 New Revision: 1714445 URL: http://svn.apache.org/viewvc?rev=1714445=rev Log: * config_store.c (serf__config_store_init): Fix bad return value. Modified: serf/trunk/config_store.c Modified: serf/trunk/config_store.c URL: http

svn commit: r1714449 - in /serf/trunk: context.c incoming.c serf.h serf_private.h test/test_all.c test/test_serf.h test/test_server.c test/test_util.c

2015-11-15 Thread rhuijben
Author: rhuijben Date: Sun Nov 15 12:52:43 2015 New Revision: 1714449 URL: http://svn.apache.org/viewvc?rev=1714449=rev Log: Start replacing a whole lot of return APR_EONOTIMPL in the serf listening and server side code with actual connection setup code, to prepare some actual http2 testing

svn commit: r1714328 - in /serf/trunk: buckets/split_buckets.c test/test_buckets.c

2015-11-14 Thread rhuijben
Author: rhuijben Date: Sat Nov 14 14:03:36 2015 New Revision: 1714328 URL: http://svn.apache.org/viewvc?rev=1714328=rev Log: Replicate some of the http2 data splitting in a test for the split buckets. (Test some event/accounting bucket behavior while we're at it) * buckets/split_buckets.c

svn commit: r1714613 - in /serf/trunk: protocols/http2_protocol.c protocols/http2_protocol.h protocols/http2_stream.c test/test_server.c

2015-11-16 Thread rhuijben
Author: rhuijben Date: Mon Nov 16 16:06:05 2015 New Revision: 1714613 URL: http://svn.apache.org/viewvc?rev=1714613=rev Log: Make the http2 protocol capable of sending a response on an incoming request. This adds basic body write support for this direction. * protocols/http2_protocol.c

svn commit: r1714622 [3/3] - in /serf/trunk: buckets/hpack_buckets.c protocols/http2_protocol.c

2015-11-16 Thread rhuijben
Modified: serf/trunk/protocols/http2_protocol.c URL: http://svn.apache.org/viewvc/serf/trunk/protocols/http2_protocol.c?rev=1714622=1714621=1714622=diff == --- serf/trunk/protocols/http2_protocol.c (original) +++

svn commit: r1714622 [2/3] - in /serf/trunk: buckets/hpack_buckets.c protocols/http2_protocol.c

2015-11-16 Thread rhuijben
Modified: serf/trunk/buckets/hpack_buckets.c URL: http://svn.apache.org/viewvc/serf/trunk/buckets/hpack_buckets.c?rev=1714622=1714621=1714622=diff == --- serf/trunk/buckets/hpack_buckets.c (original) +++

svn commit: r1714622 [1/3] - in /serf/trunk: buckets/hpack_buckets.c protocols/http2_protocol.c

2015-11-16 Thread rhuijben
Author: rhuijben Date: Mon Nov 16 16:27:04 2015 New Revision: 1714622 URL: http://svn.apache.org/viewvc?rev=1714622=rev Log: Tweak the whitespace in two more (recently added) files to be more like the other Serf files. I now configured my editor to use Serf rules in the Serf solution, so it now

svn commit: r1714598 - in /serf/trunk: incoming.c protocols/http2_protocol.c protocols/http2_protocol.h protocols/http2_stream.c serf_private.h test/test_server.c

2015-11-16 Thread rhuijben
Author: rhuijben Date: Mon Nov 16 14:39:39 2015 New Revision: 1714598 URL: http://svn.apache.org/viewvc?rev=1714598=rev Log: Add a callback to the incoming request implementation to allow enqueueing the response in a protocol dependent way. Extend http/2 incoming request handling some more. Fix

svn commit: r1714585 - /serf/trunk/protocols/http2_stream.c

2015-11-16 Thread rhuijben
Author: rhuijben Date: Mon Nov 16 13:02:06 2015 New Revision: 1714585 URL: http://svn.apache.org/viewvc?rev=1714585=rev Log: * protocols/http2_stream.c Switch file from subversion code layout to serf code layout. No functional changes... just whitespace. Modified: serf/trunk/protocols

svn commit: r1714582 - in /serf/trunk: incoming.c protocols/http2_protocol.c serf.h serf_private.h test/test_server.c

2015-11-16 Thread rhuijben
Author: rhuijben Date: Mon Nov 16 12:44:14 2015 New Revision: 1714582 URL: http://svn.apache.org/viewvc?rev=1714582=rev Log: On incoming connections: implement switching to the http2 protocol. This adds the protocol switching to the connection and http2 protocol, but doesn't add request routing

svn commit: r1714572 - /serf/trunk/outgoing.c

2015-11-16 Thread rhuijben
Author: rhuijben Date: Mon Nov 16 12:26:52 2015 New Revision: 1714572 URL: http://svn.apache.org/viewvc?rev=1714572=rev Log: * outgoing.c (serf_connection_set_framing_type): Fix behavior before socket is created. Modified: serf/trunk/outgoing.c Modified: serf/trunk/outgoing.c URL: http

svn commit: r1714594 - /serf/trunk/buckets/hpack_buckets.c

2015-11-16 Thread rhuijben
Author: rhuijben Date: Mon Nov 16 14:32:19 2015 New Revision: 1714594 URL: http://svn.apache.org/viewvc?rev=1714594=rev Log: Make the hpack buckets generate a http/1.1 style request instead of a response when the headers contain request data. * buckets/hpack_buckets.c (serf_hpack_decode_ctx_t

svn commit: r1714631 - in /serf/trunk: protocols/http2_protocol.c protocols/http2_protocol.h protocols/http2_stream.c test/test_server.c

2015-11-16 Thread rhuijben
Author: rhuijben Date: Mon Nov 16 17:12:05 2015 New Revision: 1714631 URL: http://svn.apache.org/viewvc?rev=1714631=rev Log: Add initial generic support for sending http2 request and response bodies. In theory this should make our http/2 support fully functional, although I expect that we

svn commit: r1714687 - /serf/trunk/incoming.c

2015-11-16 Thread rhuijben
Author: rhuijben Date: Mon Nov 16 21:53:31 2015 New Revision: 1714687 URL: http://svn.apache.org/viewvc?rev=1714687=rev Log: * incoming.c (includes): Add apr_portable.h. (serf__incoming_request_create): Remove unused variable. (serf__process_client): Fetch extended error, like on outgoing

svn commit: r1714653 - /serf/trunk/protocols/http2_protocol.c

2015-11-16 Thread rhuijben
Author: rhuijben Date: Mon Nov 16 19:25:26 2015 New Revision: 1714653 URL: http://svn.apache.org/viewvc?rev=1714653=rev Log: Cache pointers to a few booleans in the http2 protocol state to avoid checking for conn and client over and over again. * protocols/http2_protocol.c

svn commit: r1714679 - /serf/trunk/incoming.c

2015-11-16 Thread rhuijben
Author: rhuijben Date: Mon Nov 16 21:18:43 2015 New Revision: 1714679 URL: http://svn.apache.org/viewvc?rev=1714679=rev Log: * incoming.c (read_from_client): Don't return EAGAIN here or we exit the context loop. Modified: serf/trunk/incoming.c Modified: serf/trunk/incoming.c URL

svn commit: r1714675 - /serf/trunk/serf.h

2015-11-16 Thread rhuijben
Author: rhuijben Date: Mon Nov 16 20:57:13 2015 New Revision: 1714675 URL: http://svn.apache.org/viewvc?rev=1714675=rev Log: * serf.h (serf_incoming_request_setup_t): Fix definition. This is an out argument. Modified: serf/trunk/serf.h Modified: serf/trunk/serf.h URL: http

svn commit: r1714712 - /serf/trunk/test/serf_httpd.c

2015-11-16 Thread rhuijben
Author: rhuijben Date: Mon Nov 16 23:37:03 2015 New Revision: 1714712 URL: http://svn.apache.org/viewvc?rev=1714712=rev Log: * test/serf_httpd.c (pool_abort_func): New function. (main): Initialize pool error handling. Modified: serf/trunk/test/serf_httpd.c Modified: serf/trunk/test

svn commit: r1714711 - in /serf/trunk/protocols: http2_protocol.h http2_stream.c

2015-11-16 Thread rhuijben
Author: rhuijben Date: Mon Nov 16 23:36:29 2015 New Revision: 1714711 URL: http://svn.apache.org/viewvc?rev=1714711=rev Log: * protocols/http2_protocol.h * protocols/http2_stream.c (serf_http2__stream_handle_hpack, serf_http2__stream_handle_data): Use bool for some arguments. Modified

svn commit: r1714713 - /serf/trunk/incoming.c

2015-11-16 Thread rhuijben
Author: rhuijben Date: Mon Nov 16 23:42:03 2015 New Revision: 1714713 URL: http://svn.apache.org/viewvc?rev=1714713=rev Log: * incoming.c (read_from_client): If the initial peek fails, properly close the connection. Modified: serf/trunk/incoming.c Modified: serf/trunk/incoming.c URL

svn commit: r1714539 - in /serf/trunk: buckets/request_buckets.c serf_bucket_types.h

2015-11-16 Thread rhuijben
Author: rhuijben Date: Mon Nov 16 10:32:49 2015 New Revision: 1714539 URL: http://svn.apache.org/viewvc?rev=1714539=rev Log: Add a basic 'incoming request bucket' that can read a HTTP/1 style requests as it would be received on the server. This should be the 100% opposite of the current request

svn commit: r1714551 - /serf/trunk/buckets/request_buckets.c

2015-11-16 Thread rhuijben
Author: rhuijben Date: Mon Nov 16 11:04:40 2015 New Revision: 1714551 URL: http://svn.apache.org/viewvc?rev=1714551=rev Log: * buckets/request_buckets.c (serf_incoming_rq_parse_rqline): Following up on r1714539, parse http version, like we do in the response bucket. Modified: serf

svn commit: r1714554 - /serf/trunk/test/test_server.c

2015-11-16 Thread rhuijben
Author: rhuijben Date: Mon Nov 16 11:10:47 2015 New Revision: 1714554 URL: http://svn.apache.org/viewvc?rev=1714554=rev Log: * test/test_server.c Tweak whitespace of this recently added file to the serf standard 4 space indent. No functional changes. Modified: serf/trunk/test

svn commit: r1714546 - in /serf/trunk: deprecated.c incoming.c serf.h serf_private.h test/test_server.c

2015-11-16 Thread rhuijben
Author: rhuijben Date: Mon Nov 16 10:47:39 2015 New Revision: 1714546 URL: http://svn.apache.org/viewvc?rev=1714546=rev Log: Following up on r1714449, extend the incoming client support to basically support incoming http/1 requests. This makes the test added in the previous revision work

svn commit: r1714552 - /serf/trunk/buckets/request_buckets.c

2015-11-16 Thread rhuijben
Author: rhuijben Date: Mon Nov 16 11:06:31 2015 New Revision: 1714552 URL: http://svn.apache.org/viewvc?rev=1714552=rev Log: * buckets/request_buckets.c (serf_incoming_rq_parse_rqline): And tweak comment and error code. Modified: serf/trunk/buckets/request_buckets.c Modified: serf/trunk

svn commit: r1713936 - in /serf/trunk: buckets/allocator.c buckets/log_wrapper_buckets.c outgoing.c test/mock_buckets.c

2015-11-11 Thread rhuijben
Author: rhuijben Date: Wed Nov 11 21:29:45 2015 New Revision: 1713936 URL: http://svn.apache.org/viewvc?rev=1713936=rev Log: Resolve some SERF_DEBUG_BUCKET_USE related issues. This should make the code usable agai. * buckets/allocator.c (serf_debug__record_read): Reduce to writing an error

svn commit: r1713932 - /serf/trunk/buckets/deflate_buckets.c

2015-11-11 Thread rhuijben
Author: rhuijben Date: Wed Nov 11 21:07:58 2015 New Revision: 1713932 URL: http://svn.apache.org/viewvc?rev=1713932=rev Log: Make the deflate buckets properly read to completion in all scenarios. Fixing this made visible that the compress code calculated the crc and size in the wrong way

svn commit: r1713934 - in /serf/trunk: buckets/buckets.c serf_private.h

2015-11-11 Thread rhuijben
Author: rhuijben Date: Wed Nov 11 21:13:52 2015 New Revision: 1713934 URL: http://svn.apache.org/viewvc?rev=1713934=rev Log: Add a helper function commonly needed to fix some SERF_DEBUG_BUCKET_USE issues. * buckets/buckets.c (serf__bucket_drain): New function. * serf_private.h

svn commit: r1714775 - in /serf/trunk/protocols: http2_protocol.c http2_protocol.h

2015-11-17 Thread rhuijben
Author: rhuijben Date: Tue Nov 17 12:53:01 2015 New Revision: 1714775 URL: http://svn.apache.org/viewvc?rev=1714775=rev Log: * protocols/http2_protocol.c (serf_http2__stream_get): Use bool for arguments. * protocols/http2_protocol.h (serf_http2__stream_get): Use bool for arguments. Modified

svn commit: r1714783 - in /serf/trunk/protocols: fcgi_protocol.c fcgi_protocol.h fcgi_stream.c

2015-11-17 Thread rhuijben
Author: rhuijben Date: Tue Nov 17 13:44:24 2015 New Revision: 1714783 URL: http://svn.apache.org/viewvc?rev=1714783=rev Log: Hook up incoming requests over fcgi to the incoming client infrastructure. This bring the incoming request to the callbacks registered by the serf api user

svn commit: r1714818 - in /serf/trunk: buckets/fcgi_buckets.c protocols/fcgi_buckets.h protocols/fcgi_protocol.c protocols/fcgi_protocol.h protocols/fcgi_stream.c test/serf_httpd.c

2015-11-17 Thread rhuijben
Author: rhuijben Date: Tue Nov 17 16:07:48 2015 New Revision: 1714818 URL: http://svn.apache.org/viewvc?rev=1714818=rev Log: Start sending a response back to the fgci server. This will actually make the result visible in a webbrowser when using a httpd configured with ProxyPass "/d/fcgi/&q

svn commit: r1714774 - in /serf/trunk: buckets/fcgi_buckets.c protocols/fcgi_buckets.h protocols/fcgi_protocol.c protocols/fcgi_protocol.h protocols/fcgi_stream.c

2015-11-17 Thread rhuijben
Author: rhuijben Date: Tue Nov 17 12:49:11 2015 New Revision: 1714774 URL: http://svn.apache.org/viewvc?rev=1714774=rev Log: Extend fcgi code with parameter decoding. This completes the server side request parsing. Next step will be to hook up the incoming request routing to get us a response

svn commit: r1714792 - /serf/trunk/buckets/chunk_buckets.c

2015-11-17 Thread rhuijben
Author: rhuijben Date: Tue Nov 17 14:26:34 2015 New Revision: 1714792 URL: http://svn.apache.org/viewvc?rev=1714792=rev Log: * buckets/chunk_buckets.c (serf_chunk_peek): Peek should never return EAGAIN. Modified: serf/trunk/buckets/chunk_buckets.c Modified: serf/trunk/buckets

svn commit: r1714781 - /serf/trunk/buckets/headers_buckets.c

2015-11-17 Thread rhuijben
Author: rhuijben Date: Tue Nov 17 13:40:01 2015 New Revision: 1714781 URL: http://svn.apache.org/viewvc?rev=1714781=rev Log: Fix a very old bug in the header buckets readline that we somehow never noticed in our previous uses of this bucket. Reading the header buckets as an incoming request

svn commit: r1711741 - /serf/trunk/test/test_util.c

2015-11-01 Thread rhuijben
Author: rhuijben Date: Sun Nov 1 12:36:05 2015 New Revision: 1711741 URL: http://svn.apache.org/viewvc?rev=1711741=rev Log: * test/test_util.c (test_setup): Only keep 16 MB of unused memory during testing. (Subversion defaults to 4 MB). The default was to never

svn commit: r1711984 - /serf/trunk/buckets/copy_buckets.c

2015-11-02 Thread rhuijben
Author: rhuijben Date: Mon Nov 2 13:46:40 2015 New Revision: 1711984 URL: http://svn.apache.org/viewvc?rev=1711984=rev Log: Simplify some code in the copy bucket code and avoid copying data (and allocating the buffer) in a not unlikely scenario. * buckets/copy_buckets.c (serf__copy_iovec

svn commit: r1712102 - /serf/trunk/buckets/bwtp_buckets.c

2015-11-02 Thread rhuijben
Author: rhuijben Date: Mon Nov 2 18:09:16 2015 New Revision: 1712102 URL: http://svn.apache.org/viewvc?rev=1712102=rev Log: * buckets/bwtp_buckets.c (serialize_data): Properly prepare argument printed with APR_UINT64_T_HEX_FMT. Modified: serf/trunk/buckets/bwtp_buckets.c Modified

svn commit: r1712065 - /serf/trunk/protocols/http2_protocol.c

2015-11-02 Thread rhuijben
Author: rhuijben Date: Mon Nov 2 16:16:10 2015 New Revision: 1712065 URL: http://svn.apache.org/viewvc?rev=1712065=rev Log: Remove some * protocols/http2_protocol.c (serf_bucket_create_numberv): Increase code compatibility by not calling va_arg() with a second argument smaller than int

svn commit: r1712098 - /serf/trunk/build/check.py

2015-11-02 Thread rhuijben
Author: rhuijben Date: Mon Nov 2 17:50:29 2015 New Revision: 1712098 URL: http://svn.apache.org/viewvc?rev=1712098=rev Log: * build/check.py Print a bit more information about the failed test. The returncode might be interesting to track down signal problems. Modified: serf/trunk/build

svn commit: r1712099 - /serf/trunk/protocols/http2_protocol.c

2015-11-02 Thread rhuijben
Author: rhuijben Date: Mon Nov 2 17:55:39 2015 New Revision: 1712099 URL: http://svn.apache.org/viewvc?rev=1712099=rev Log: * protocols/http2_protocol.c (serf_bucket_create_numberv): Resolve two more warnings. Modified: serf/trunk/protocols/http2_protocol.c Modified: serf/trunk/protocols

svn commit: r1712131 - /serf/trunk/SConstruct

2015-11-02 Thread rhuijben
Author: rhuijben Date: Mon Nov 2 20:01:59 2015 New Revision: 1712131 URL: http://svn.apache.org/viewvc?rev=1712131=rev Log: * SConstruct Let scons generate the valid options. Add the most likely next version of Visual Studio to the list. Modified: serf/trunk/SConstruct Modified: serf

svn commit: r1712271 - /serf/trunk/buckets/buckets.c

2015-11-03 Thread rhuijben
Author: rhuijben Date: Tue Nov 3 12:15:06 2015 New Revision: 1712271 URL: http://svn.apache.org/viewvc?rev=1712271=rev Log: Remove accidental garbage committed in r1712270. * buckets/buckets.c (it): Remove. Modified: serf/trunk/buckets/buckets.c Modified: serf/trunk/buckets/buckets.c

svn propchange: r1712270 - svn:log

2015-11-03 Thread rhuijben
Author: rhuijben Revision: 1712270 Modified property: svn:log Modified: svn:log at Tue Nov 3 12:16:22 2015 -- --- svn:log (original) +++ svn:log Tue Nov 3 12:16:22 2015 @@ -5,5 +5,8 @@ Fix some testsuite issues when

svn propchange: r1711798 - svn:log

2015-11-01 Thread rhuijben
Author: rhuijben Revision: 1711798 Modified property: svn:log Modified: svn:log at Sun Nov 1 15:09:26 2015 -- --- svn:log (original) +++ svn:log Sun Nov 1 15:09:26 2015 @@ -2,6 +2,10 @@ Hugely reduce the number of reads

svn commit: r1711772 - in /serf/trunk: buckets/http2_frame_buckets.c protocols/http2_buckets.h protocols/http2_protocol.c test/test_buckets.c

2015-11-01 Thread rhuijben
Author: rhuijben Date: Sun Nov 1 14:27:51 2015 New Revision: 1711772 URL: http://svn.apache.org/viewvc?rev=1711772=rev Log: Cleanup some constructor arguments of http2 decompression buckets. Destroying the stream with an unframe bucket doesn't make sense and not destroying the frame

svn commit: r1711824 - /serf/trunk/buckets/http2_frame_buckets.c

2015-11-01 Thread rhuijben
Author: rhuijben Date: Sun Nov 1 16:17:04 2015 New Revision: 1711824 URL: http://svn.apache.org/viewvc?rev=1711824=rev Log: * buckets/http2_frame_buckets.c (serf_http2_unpad_read_iovec): Decrement value instead of pointer. Found by: clang Modified: serf/trunk/buckets

svn commit: r1711798 - /serf/trunk/buckets/response_body_buckets.c

2015-11-01 Thread rhuijben
Author: rhuijben Date: Sun Nov 1 15:02:14 2015 New Revision: 1711798 URL: http://svn.apache.org/viewvc?rev=1711798=rev Log: Hugely reduce the number of reads necessary to drain a HTTP/1.1 content-length limited response body, by directly returning EOF when we hit the end of the body

svn commit: r1711822 - in /serf/trunk/protocols: http2_buckets.h http2_protocol.c http2_stream.c

2015-11-01 Thread rhuijben
Author: rhuijben Date: Sun Nov 1 16:08:39 2015 New Revision: 1711822 URL: http://svn.apache.org/viewvc?rev=1711822=rev Log: Resolve some warnings found by compiling using clang on FreeBSD. * protocols/http2_buckets.h (serf_hpack_table_t): Remove duplicated definition. * protocols

svn commit: r1711825 - /serf/trunk/buckets/request_buckets.c

2015-11-01 Thread rhuijben
Author: rhuijben Date: Sun Nov 1 16:19:06 2015 New Revision: 1711825 URL: http://svn.apache.org/viewvc?rev=1711825=rev Log: * buckets/request_buckets.c (includes): Following up on r1709439, add serf_private.h to match prototype against implementation. Modified: serf/trunk/buckets

svn commit: r1711794 - /serf/trunk/buckets/iovec_buckets.c

2015-11-01 Thread rhuijben
Author: rhuijben Date: Sun Nov 1 14:56:57 2015 New Revision: 1711794 URL: http://svn.apache.org/viewvc?rev=1711794=rev Log: Implement get_remaining support on the standard iovec bucket. * buckets/iovec_buckets.c (serf_iovec_get_remaining): New function. (serf_bucket_type_iovec): Implement

svn commit: r1711823 - in /serf/trunk: buckets/hpack_buckets.c protocols/http2_protocol.h

2015-11-01 Thread rhuijben
Author: rhuijben Date: Sun Nov 1 16:14:09 2015 New Revision: 1711823 URL: http://svn.apache.org/viewvc?rev=1711823=rev Log: Following up on r1711822, fix an error and a few more warnings. * buckets/hpack_buckets.c (serf_bucket_hpack_getc): Rename to... (serf__bucket_hpack_getc

svn commit: r1712641 - /serf/trunk/buckets/mmap_buckets.c

2015-11-04 Thread rhuijben
Author: rhuijben Date: Wed Nov 4 20:48:57 2015 New Revision: 1712641 URL: http://svn.apache.org/viewvc?rev=1712641=rev Log: Fix a bunch of warnings by storing the remaining size of a memory mapped region as apr_size_t instead of apr_off_t. * buckets/mmap_buckets.c (mmap_context_t): Make

svn commit: r1712648 - /serf/trunk/buckets/mmap_buckets.c

2015-11-04 Thread rhuijben
Author: rhuijben Date: Wed Nov 4 20:58:05 2015 New Revision: 1712648 URL: http://svn.apache.org/viewvc?rev=1712648=rev Log: Following up on r1712641, add some error checking and revert some accidentally removed code that had multiple functions. * buckets/mmap_buckets.c (serf_mmap_read): Add

svn commit: r1712694 - /serf/trunk/connection_request.c

2015-11-04 Thread rhuijben
Author: rhuijben Date: Thu Nov 5 00:29:38 2015 New Revision: 1712694 URL: http://svn.apache.org/viewvc?rev=1712694=rev Log: Combine a bit of cleanup code for explicit cleanup and that via the cleanup pools. * connection_request.c (clean_resp): Introduce a bit of cleanup code here

svn commit: r1712435 - in /serf/trunk: buckets/hpack_buckets.c buckets/prefix_buckets.c test/test_buckets.c

2015-11-03 Thread rhuijben
Author: rhuijben Date: Tue Nov 3 23:09:09 2015 New Revision: 1712435 URL: http://svn.apache.org/viewvc?rev=1712435=rev Log: Resolve two memory leaks found by ivan. While looking at this code also shrink the amount of memory needed at runtime while decoding http2 hpack data. * buckets

svn commit: r1712621 - in /serf/trunk: buckets/aggregate_buckets.c outgoing.c serf_bucket_types.h ssltunnel.c

2015-11-04 Thread rhuijben
Author: rhuijben Date: Wed Nov 4 19:51:54 2015 New Revision: 1712621 URL: http://svn.apache.org/viewvc?rev=1712621=rev Log: In preparation for really fixing the root cause revert the ostream_tail creation to use a normal aggregate stream which does destroy the buckets added to it when done

svn commit: r1712628 - in /serf/trunk: connection_request.c outgoing.c

2015-11-04 Thread rhuijben
Author: rhuijben Date: Wed Nov 4 20:13:34 2015 New Revision: 1712628 URL: http://svn.apache.org/viewvc?rev=1712628=rev Log: Following up on r1712622, consistently assume that the bucket is now owned by the output stream as soon as writing_started is set. * connection_request.c (clean_resp

  1   2   3   4   >