kc3-lang/libevent

Branch :


Log

Author Commit Date CI Message
7bcace2d 2010-11-22 21:02:34 Fix some irix compilation warnings spotted by Kevin Bowling
150d7d0a 2010-11-22 20:24:03 Merge remote branch 'kev009/master'
e8749823 2010-11-22 16:40:31 Fix IRIX build. sa_family collides with a #define in sys/socket.h on IRIX.
26049c2f 2010-11-22 16:24:52 Merge remote branch 'github/20_getaddrinfo_cancel_v2'
da1bf528 2010-11-22 16:24:09 Add a stress test for getaddrinfo_cancel
568ac4fd 2010-11-22 15:52:34 Merge remote branches 'github/20_epoll_nochangelist_v4', 'github/20_openssl_closeonfree' and 'github/20_cloexec'
aed7e029 2010-11-22 14:10:01 Make unit tests for epoll-with-changelist pass The only changes needed were to handle the fact that the methodname "epoll (with changelist)" matches the environment variable EVENT_NOEPOLL rather than the imaginary "EVENT_EPOLL (WITH CHANGELIST)".
ece974fb 2010-11-22 14:13:24 Reorder backends in test.sh to match preference order in event.c
9531763a 2010-11-14 17:52:16 Disable changelist for epoll by default because of Linux dup() bug; add an option and/or an envvar to reenable it for speed. Rename option to control epoll changelist; make epoll changelist off by default
3ab578f8 2010-11-20 01:41:34 Make sure the CLOEXEC flag is set on fds we open for base notification
3a67d0bf 2010-11-19 17:09:30 Resolve an evport bug in the thread/forking test
d51b2fc6 2010-11-19 12:14:18 Make evdns_getaddrinfo_cancel threadsafe
c7cfbcf4 2010-11-19 12:01:05 Fix some more cancel-related bugs in getaddrinfo_async Also imposed a new rule to make this much much simpler: no freeing the getaddrinfo request until both dns callbacks have been invoked.
abf01ed1 2010-11-19 11:33:48 Avoid double-invocation of user callback with EVUTIL_EAI_CANCEL
49418612 2010-11-16 12:55:10 Use the US-English "canceled", not the UK "cancelled".
8faf223a 2010-11-17 00:09:10 Fix a memory leak in evhttp_uri_free.
4f228a1f 2010-11-14 19:52:18 Fix bug in bufferevent_connect on an openssl bufferevent that already had an fd The problem was that we were using openssl's BIO code's shutdown flag whenever BEV_OPT_CLOSE_ON_FREE was set. This made the BIO close the socket when it was freed... but it would be freed whenever we did a setfd on the bufferevent_openssl, even the no-op setfd in bufferevent_connect. So instead, we just set the shutdown flag to 0, and handle closing the fd ourselves. Spotted by Linus Nordberg
1ac5b230 2010-11-09 15:18:59 Only clear underlying callbacks when the user hasn't reset them.
fc7b1b00 2010-11-09 11:43:47 When closing a filtering bufferevent, clear callbacks on the underlying bufferevent Previously, if BEV_OPT_CLOSE_ON_FREE wasn't set on a bufferevent_filter or a filtering bufferevent_openssl, when we went to free the filtering bufferevent, we'd leave the underlying bufferevent unchanged. That's not so good, since the callbacks are set to activate stuff in the filtering bufferevent that we're about to free. Instead, set all the callbacks to NULL.
5c8a59e8 2010-11-09 10:19:05 Merge remote branches 'github/20_evdns_cancel_segfault_v2', 'github/20_http_close_detect', 'github/20_http_versions', 'github/20_more_http_methods', 'github/20_shutdown_iocp_listener' and 'github/20_win64_fixes'
a38140be 2010-11-09 10:14:32 Refactor http version parsing into a single function Based on a suggestion by Chris Davis to make evhttp_parse_response_line tolerate odd versions too.
3db6bc00 2010-11-09 10:03:00 Remove some debugging puts() calls from allow_methods test
52aa419b 2010-11-04 15:40:44 Set SO_UPDATE_ACCEPT_CONTEXT on sockets from AcceptEx so that shutdown() can work Based on patch (and lots of debugging work) by Kelly Brock.
229714d1 2010-11-04 16:04:28 Fix a mistake in http documentation found by Julien Blache
05124879 2010-11-04 14:05:08 Never call evhttp_readcb while writing.
c76640b5 2010-06-01 04:45:55 Don't disable reading from the HTTP connection after sending the request to be notified of connection-close in time
75e3320e 2010-11-04 12:41:13 Units test for unexpected evhttp methods.
536311a4 2010-11-04 12:39:41 evhttp: Return 501 when we get an unrecognized method, not 400.
f5b391e2 2010-11-04 11:53:34 Tweak interface for allowed methods
75a73414 2010-11-04 11:25:35 Define enumerators for all HTTP methods, including PATCH from RFC5789 This patch defines enumerators for all HTTP methods that exist (including PATCH introduced in RFC 5789). It also makes them bit-masky (that's not a word, is it?), breaking binary- but not source-code compatibility. evhttp now stores a bitmask specifying for which methods requests to dispatch and which ones to reject with "405 Method Not Allowed". By default that's the ones we currently have (GET, POST, HEAD, PUT, DELETE), thereby keeping functional compatibility (besides the minor change that one of the other methods will now cause 405 instead of 400. But I believe that could even be considered a bug-fix). evhttp is extended by evhttp_set_allowed_methods() with which the user can change that bitmask. no regressions here and my test-app still works. Haven't yet actually tested any of the new methods. What's obviously missing here is the special logic for the methods: OPTIONS: We should be fine here - I believe our current dispatch logic should work fine. Some convenience functions would be fine though. TRACE: I'm pretty certain we should never dispatch this to the callbacks and simply implement the necessary functionality built-in. CONNECT: Pretty straight-forward to implement (and considering the framework in which we implement it very efficient too). Should probably go built-in. PATCH: Except for checking the RFC against our pre-dispatch logic (there just might be some "MUST not have Some-Header" lurking somewhere) there is nothing to be done here, this is completely up to the user. Nothing to do.
9ed30de7 2010-11-03 12:37:37 Don't free evdns_request handles until after the callback is invoked Previously, once the callback was scheduled, it was unsafe to cancel a request, but there was no way to tell that. Now it is safe to cancel a request until the callback is invoked, at which point it isn't. Found and diagnosed by Denis Bilenko.
34b84b97 2010-11-03 14:38:45 Fix more wn64 warnings.
78762383 2010-11-03 15:18:34 Merge branch 'http_nolegacy_v2'
985430ae 2010-11-03 15:17:57 Remove need for http_compat.h in http tests
0b137f45 2010-11-03 15:14:29 Stop accessing http request struct directly from in the unit tests.
22e0a9b2 2010-11-03 15:12:08 Add evhttp_response_code to remove one more reason to include http_struct.h
c91622d1 2010-11-03 15:04:44 Stop using event_compat.h in regress_http
9bb82393 2010-11-03 14:31:23 Convert the rest of the http tests to be non-legacy unit tests.
353402a8 2010-11-03 14:13:20 Rename the confusing "base" static variable in regress_http.c
8505a744 2010-11-03 13:55:20 Start porting http tests to not use legacy interfaces
647e094c 2010-11-02 15:19:12 Replace exact-version checks for HTTP/1.1 with >= or < checks
aa5f55fa 2010-11-02 13:50:57 reset "chunked" flag when sending non-chunked reply
ba014569 2010-11-02 12:42:35 Use the label_len local variable in evdns instead of recalculating it over and over
a3245afe 2010-11-01 14:23:33 Fix win32 build in response to fixes from win64 build.
74a91e5a 2010-11-01 14:16:39 fix signed/unsigned warnings in http.c
f8095d64 2010-11-01 14:15:34 Fix a typo in 7484df61c981fc33db2~
545a6114 2010-11-01 13:59:04 Fix even more win64 warnings: buffer, event_tagging, http, evdns, evrpc
7484df61 2010-11-01 13:43:43 Fix even more win64 warnings
b6a158ca 2010-11-01 11:48:57 Rename "size" variables in win32select that were really fd counts.
f8064762 2010-10-28 10:11:25 Increase the skew tolerance to 2 seconds in thread/deferred_cb_skew This is to make the test not fail on Sebastian Hahn's Win7 box.
7b40a000 2010-10-28 10:08:17 Make sure IOCP evconnlistener uses virtual events.
598d1336 2010-10-27 22:57:53 Try to clear up more size_t vs int/long issues.
b81217f7 2010-10-27 17:37:32 Fix signal handler types for win64.
1ae82cd8 2010-10-27 17:32:41 Set _EVENT_SIZEOF_VOID_P correctly on win32 and win64
f817bfa4 2010-10-27 17:31:52 Fix some ints to evutil_socket_t; make tests pass on win64.
19c71e74 2010-10-27 10:36:08 Fix som event_warns that should have been event_warnx
e8a903ce 2010-10-27 10:27:04 Merge remote branch 'trondn/master'
f5ad31c1 2010-10-27 12:47:07 Check return value for ioctlsocket on win32
5d389dc0 2010-10-26 22:27:57 Fix some uses of int for socket in regress
84a7053e 2010-10-26 21:33:22 Merge remote branch 'github/20_http_read_after_write'
73bf07fe 2010-10-26 21:33:13 Merge remote branch 'github/20_abi_breaks'
093fb989 2010-10-26 21:33:05 Merge remote branch 'github/20_ratelim_size'
4f20eeaa 2010-10-26 21:32:53 Merge remote branch 'github/20_chain_realign'
fbaf0770 2010-10-26 12:09:20 Fix bugs in posix thread-id calculation when sizeof(pthread_t) != sizeof(long) When pthread_t was smaller, our calculated thread IDs would include uninitialized RAM, and so our unit tests would fail because thread_ids would never match one another. When pthread_t was larger and alignment was big-endian, our calculated thread IDs would only have the most significant bytes of the pthread_t, when in practice all the entropy is in the low-order bytes. Found with help from Dagobert Michelsen.
ac1931ac 2010-10-26 11:07:26 Remove event-config.h from .gitignore; it moved to include/event2
e56ff65a 2010-10-26 11:01:58 Fix a minor syntax error that most compilers didn't care about
a4063c06 2010-10-26 10:38:30 Note that 2.0.9 will break the ABI, and make changes we were postponing. We had to turn a couple of 32-bit size arguments into 64-bit arguments or size_t arguments (since otherwise we would have had to do it post 2.0.x-stable, and that would be worse).
2cbb1a16 2010-10-26 10:27:29 Make rate-limits go up to SIZE_MAX/EV_SSIZE_MAX, not just INT32_MAX Someday, when networks are far faster and people frequently want a burst value greater than 2GB per tick, this will seem very forsightful indeed. For now, it breaks ABI, but not source. Fixes bug 3092096.
e4f34e8a 2010-10-25 22:36:23 Correct logic for realigning a chain in evbuffer_add The old logic was both too eager to realign (it would move a whole chain to save a byte) and too reluctant to realign (it would only realign when data would fit into the misaligned portion, without considering the space at the end of the chain). The new logic matches that from evbuffer_expand_singlechain: it only realigns a chain when not much data is to be moved, and there's a bunch of space to be regained. Spotted by Yan Lin.
74c0e862 2010-10-25 21:53:15 Avoid missed-request bug when entire http request arrives before data is flushed The trigger for starting to read the first line of a request used to be, "When data has arrived and we're looking for the first line." But that's not good enough: if the entire next request gets read into our bufev->inbuf while we're still processing the current request, we'll never see any more data arrive, and so will never process it. So the fix is to make sure that whenever we hit evhttp_send_done, we call evhttp_read_cb. We can't call it directly, though, since evhttp_send_done is reachable from the user API, and evhttp_read_cb can invoke user functions, and we don't want to force everyone to have reentrant callbacks. So, we use a deferred_cb. Found by Ivan Andropov. This is bug 3008344.
8e342e56 2010-10-25 16:09:11 Correctly count req->body_size on http usage without Content-Length There was a dumb bug where we would look at the length of the input buffer immediately _after_ we drained it.
58a1cc6b 2010-10-25 16:00:47 Fix a bug where we would read too much data in HTTP bodies or requests. We were using evbuffer_add_buffer, which moved the entire buffer contents. But if we had a valid content_length, we only wanted to move up to the amount of data remaining in ntoread. Our bug would make us put our ntoread in the negative, which would in turn make us read all data until the connection closed. Found by Denis Bilenko. Should fix bug 2963172.
525da3e1 2010-10-25 15:49:42 Fix Content-Length when trying send more than 100GB of data (!) on an evhttp.
f1250eb6 2010-10-25 15:23:41 add a requested docstring for event_rpcgen.CommandLine.__init__
9c71a341 2010-10-25 15:13:32 Merge remote branch 'github/http_and_listener'
ac7e52d8 2010-10-25 14:29:30 Make evbuffer_add_file take ev_off_t, not off_t This change has no effect on non-windows platforms, since those either define off_t to 64-bits, or allow you to decide whether it should be 64-bits yourself via some LARGEFILE-like macro. On Windows, however, off_t is always 32-bit, so it's a bad choice for "file size" or "file offset" values. Instead, I'm adding an ev_off_t type, and using it in the one place where we used off_t to mean "the size of a file" or "an offset into a file" in the API. This breaks ABI compatibility on Windows.
006efa7d 2010-10-25 11:50:51 Functions to actually use evhttp_bound_socket with/as evconnlistener.
46ee061c 2010-10-25 11:47:05 Add a function to change a listener's callback. You can also now initialize listeners with no callbacks set; if so, they won't get enabled until the callback is set to non-NULL.
2c66983a 2010-10-24 11:51:14 Simplify the logic for choosing EPOLL_CTL_ADD vs EPOLL_CTL_MOD Previously, we chose "ADD" whenever old_events==new_events, (since we expected the add to fail with EEXIST), or whenever old_events was==0, and MOD otherwise (i.e., when old_events was nonzero and not equal to new_events). But now that we retry failed MOD events as ADD *and* failed ADD events as MOD, the important thing is now to try to guess right the largest amount of the time, since guessing right means we do only one syscall, but guessing wrong means we do two. When old_events is 0, ADD is probably right (unless we're hitting the dup bug, when we'll fall back). And when old_events is set and != new_events, MOD is almost certainly right for the same reasons as before. But when old_events is equal to new events, then MOD will work fine unless we closed and reopened the fd, in which case we'll have to fall back to the ADD case. (Redundant del/add pairs are more common than closes for most use cases.) This change lets us avoid calculating new_events, which ought to save a little time in epoll.c
c281aba3 2010-10-24 11:38:29 Fix a nasty bug related to use of dup() with epoll on Linux Current versions of the Linux kernel don't seem to remove the struct epitem for a given (file,fd) combo when the fd is closed unless the file itself is also completely closed. This means that if you do: fd = dup(fd_orig); add(fd); close(fd); dup2(fd_orig, fd); add(fd); you will get an EEXIST when you should have gotten a success. This could cause warnings and dropped events when using dup and epoll. The solution is pretty simple: when we get an EEXIST from EPOLL_CTL_ADD, we retry with EPOLL_CTL_MOD. Unit test included to demonstrate the bug. Found due to the patient efforts of Gilad Benjamini; diagnosed with help from Nicholas Marriott.
bf11e7dd 2010-10-21 15:33:13 Merge branch 'http_uri_parse'
bc98f5e6 2010-10-21 14:53:21 Unit tests for evhttp_uri_set*
45f6869c 2010-10-21 14:41:12 Make evhttp_uri non-public, and give it accessor functions.
70e1b607 2010-10-21 14:05:04 Document that two bufferevent functions only work on socket bufferevents
aab49b60 2010-10-21 14:04:24 Add a bufferevent_get_base function
d9ffa899 2010-10-21 12:48:13 Update the HTTP regression tests to use Libevent2 apis for non-http stuff
1f507d75 2010-10-21 12:27:16 Stop using Libevent-1 headers in regress_http
2a3b5872 2010-10-21 12:23:10 Merge branch 'http_small_tweaks' Conflicts: http-internal.h
cd00079b 2010-10-21 12:19:28 Add evhttp_connection_get_base() to get the event_base from an http connection Based on a patch by Mark Ellzey from 27 July 2010. Closes ticket 3052406
1213d3dd 2010-10-20 13:41:02 Fix a 100%-CPU bug where an SSL connection would sometimes never stop trying to write If an SSL connection becamse disabled or suspended before became open, it could (under the right circumstances) wind up without ever getting its write callback disabled. The most correct fix is probably more subtle, and involves checking all caseswhen a write callback is enabled or disabled. This fix is more blunt, and explicitly checks whether the callback should have been disabled at the end of the callback to prevent infinite looping. Diagnosed with help from Sebastian Hahn
2075fbcf 2010-10-19 13:15:48 Add evhttp_parse_query_str to be used with evhttp_uri_parse. The old evhttp_parse_query() doesn't work well with struct evhttp_uri.query, since it expects to get whole URIs, rather than just the query portion.
3a334628 2010-10-19 13:02:18 Document behavior of URI parsing more thoroughly. Also, move evhttp_uri struct into http.h, since it is part of the API.
a5a76e68 2010-10-19 12:35:50 Add a huge pile of tests for the new URI functions, and make them pass.
ad923a11 2010-10-19 12:33:50 Improvements to tinytest_macros.h First, handle cases where we have %s in a tt_want or tt_assert. Second, add tt_want_*_op that do a tt_*_op test, but do not exit the test on failure. We should push these upstream to tinytest some time.
eaa5f1d9 2010-10-19 11:26:59 Revise evhttp_uri_parse implementation to handle more of RFC3986
fadbfd4e 2010-10-18 14:43:54 Clean up error handling in uri_parse a little
7d45431e 2010-10-18 14:38:48 Do not silently truncate URIs in evhttp_uri_join. Also avoid evbuffer_pullup.
86212341 2010-10-18 14:34:20 Make evhttp_uri_parse and friends conform to memory management standards
86dd720a 2010-08-08 16:46:39 Introduce absolute URI parsing helpers. See evhttp_uri_parse(), evhttp_uri_free() and evhttp_uri_join() for details.
f13e449b 2010-10-18 14:20:06 Merge branch 'http_parse'