Avoid setting IOStream.error to wrong exception, close #651

Replace a couple of contextmanager functions with ExceptionStackContext.

Merge remote-tracking branch 'rhettg/master' into work

Run AsyncHTTPClient callbacks on the IOLoop for a clean stack.

Closes #652.

TwistedIOLoop now supports overriding handle_callback_exception.

Remove some now-unnecessary calls to functools.partial

Add forgotten release note.

add_callback now takes *args, **kwargs.

This reduces the need for functools.partial or lambda wrappers, and
works better with stack_context in some cases since binding the
arguments within IOLoop lets it see whether the function is already
wrapped.

Don't catch and hide exceptions caused by callbacks in http client

curl_httpclient: don't call remove_handler if we never added it.

Re-order IOLoop and self._fds operations for consistency.

Catch all exceptions, not just {OS,IO}Error in IOLoop.remove_handler.

Attempting to remove a non-existent fd raises IOError on epoll
but KeyError on kqueue; this change swallows any exeption to make
both platforms consistent.

Add path_{kw,}args attributes to RequestHandler.

Return a Vary: Accept-Encoding header whenever gzip is enabled.

This is one of two problems found with redbot.  The other,
that etags should change when content-encoding is used, is trickier to
fix and seems to be less problematic.

Closes #578.

Add tests using the Redbot library for HTTP validation and caching.

Document the use of self.xsrf_token to set the cookie for ajax-only apps.

Closes #645.

Document the need to pass Connection: close when using no_keep_alive.

Closes #641.

Add a mock.patch-compatible wrapper for options objects.

Doc cleanups.

* Fix doc generation for httpclient.py (unescaped \r\n)
* Manually specify signature of Template constructor
* Include IOLoop.time

Closes #644.

ErrorHandler no longer requires XSRF tokens.

Closes #638.

Add protocol kwarg to HTTPServer constructor.

Accept Content-Length headers in 304 responses.

Closes #639.

Remove now-redundant entries from next.rst, bump version to 2.4.post2.

Merge branch 'branch2.4'

Conflicts:
setup.py
tornado/__init__.py
tornado/test/auth_test.py
website/sphinx/releases.rst

Bump version to 2.4.1, update release notes.

Fix a bug with IOStream.read_until_close with a streaming_callback.

If the underlying socket was already closed the buffered data
would be passed to the final callback rather than the streaming callback.

Conflicts:
website/sphinx/releases/next.rst

Fix TwitterMixin on Python 3.

Also add tests, and add get_auth_http_client method to all auth mixins.

Closes #634.

Conflicts:
tornado/test/auth_test.py
website/sphinx/releases/next.rst

Fix a memory leak in stack_context.

The old_contexts reference in StackContexts could maintain a chain of
old irrelevant contexts, so clear it once it's no longer needed.
This was mainly a problem in gen.engine, where additional contexts
would accumulate in memory (but not on the stack) for each asynchronous
operation.

Also clear the deactivate_stack_context in gen.Runner to allow
the StackContext to be garbage-collected sooner.

Use AsyncHTTPClient's defaults option in AsyncHTTPSTestCase.

Fix test_non_ssl_request, which has apparently been testing the wrong
thing for some time.

Remove redundant logging of read errors in IOStream.

The logging in read_from_fd is generally redundant with logging at
higher levels - the error would be logged again in _handle_read,
or propagated from _try_inline_read.

Closes #632.

Allow default HTTPRequest attributes to be set globally via configure.

Closes #379.

Remove outdated warnings and config instructions from simple_httpclient.

Merge branch 'branch2.4'

Conflicts:
website/sphinx/database.rst

Upgrade appengine website to python 2.7.

Update app engine id for HRD migration.

Document the deprecation of tornado.database in the 2.4 branch.

Make header_callback behavior consistent across both HTTP clients.

SimpleAsyncHTTPClient now sends the first and last lines to the callback,
matching CurlAsyncHTTPClient's behavior.

All secondary callbacks are now wrapped for stack_context.

Closes #637.

Fix If-None-Match support for StaticFileHandler.

No longer set Cache-Control: public on static files; redbot.org says
it generally doesn't do anything useful.

Fix a memory leak in stack_context.

The old_contexts reference in StackContexts could maintain a chain of
old irrelevant contexts, so clear it once it's no longer needed.
This was mainly a problem in gen.engine, where additional contexts
would accumulate in memory (but not on the stack) for each asynchronous
operation.

Also clear the deactivate_stack_context in gen.Runner to allow
the StackContext to be garbage-collected sooner.

Ensure that add_callback fails cleanly if called while IOLoop is closing.

Previously there would (sometimes) be an exception when writing to the
waker pipe.

Closes #635.

Fix TwitterMixin on Python 3.

Also add tests, and add get_auth_http_client method to all auth mixins.

Closes #634.

Merge pull request #627 from jparise/urlspec-repr

Add a helpful __repr__ implementation to URLSpec.

Merge pull request #624 from pablopda/master

Move README so github web interface recognize its markup language

Fix a bug with IOStream.read_until_close with a streaming_callback.

If the underlying socket was already closed the buffered data
would be passed to the final callback rather than the streaming callback.

Add a helpful __repr__ implementation to URLSpec.

Mv README so github web interface recognize its markup language

Release notes for 302/303 change.

simple_httpclient: Treat 302 like 303.

Cherry-picked from https://github.com/bergundy/tornado/commit/e49b263de2f4aa7b9748531b2ccf6b50b28b1bd5

Closes #623.

AsyncHTTPTestCase no longer closes AsyncHTTPClients on the global IOLoop.

Code that uses the global IOLoop may retain a global reference to
an AsyncHTTPClient as well, which would become inoperable after a test
had closed it.

Added more sensical callback time verification.

Added period check in PeriodicCallback

If a user currently passes in 0ms (hopefully by accident) as the
callback time for a periodic callback, Tornado consumes a huge amount
of CPU and never calls the function.

Add a comment to runtests.sh about python 3.

Closes #612.

Actually use add_callback_from_signal from the SIGCHLD handler.

Fix TwistedIOLoop on epoll.

Twisted's removeAll doesn't like it if a socket is closed without being
unregistered.  EPoll generates events slightly differently so be more
careful about doubled connectionLost events.

Implement TwistedIOLoop, to bridge the gap with Twisted in the other direction.

This also serves as a proof of concept for the refactored IOLoop interface.

Remove the method IOLoop.running.

Split IOLoop into a base class and poll-based implementation.

The base class has all the methods with docstrings; nearly all the
implementation is in a new subclass PollIOLoop.

Move IOLoop subclasses to their own modules under platform.

Add a little more slack to this timing test.

It seems to fail a lot on VMs.

Check self.closed() in the inner read loop of IOStream.

This was causing error logs in the keepalive tests (but only on epoll)

Avoid creating multiple redundant autoreload callbacks.

One more log spam test: connect then immediately close the connection.

Add tests and release notes for {% apply %} unicode fix.

Merge remote-tracking branch 'dmitric/master'

Fix typo in release notes (thanks jparise!))

Ensure result from apply block is utf8

Reduce log spam from closed client connections.

Added a bunch of tests for keepalive functionality and fixed two cases
where we'd log an exception when the client was gone.  ECONNRESET
errors in IOStream reads now just close the connection instead of
logging an error (the exception information is still available on
stream.error in the close callback for apps that want it).
HTTPConnection now also checks for a closed connection and cleans up
instead of logging an error.

IOStream now raises a new exception class StreamClosedError instead of
IOError.

Add time_func parameter to IOLoop, and make it possible to use time.monotonic.

This means that calls to IOLoop.add_timeout that pass a number must be
updated to use IOLoop.time instead of time.time.

There are still some places where we use time.time in the code, but they
are either places where wall time is desired, or non-critical deltas (e.g.
printing elapsed time at the end of a request).

Thanks to apenwarr and mgenti for pull requests and discussion relating to
this change. (#558 and #583)

IOLoop poller implementations are now subclasses of IOLoop.

Update testing command-line flags to allow configuration of a non-default
IOLoop.

Extract configure logic from AsyncHTTPClient to a base class.

IOLoop now extends this base class as well, although no other
implementations are provided yet.  This does not include the
pseudo-singleton magic from AsyncHTTPClient.

AsyncHTTPClient: remove legacy magic around max_clients arg.

Fix a bug in which configured kwargs would be applied to a non-configured
class.

OptionParser now contains a dict rather than subclassing dict.

Doc updates for the options module

Rename _Options to OptionParser, and fix bugs with --help

Add per-option callbacks, and implement --help with them.

Introduce parse callbacks in the options module.

This is a generalization of the existing magic for logging.  Logging
configuration is now moved to log.py.

AsyncHTTPTestCase: bind the server port earlier so it's available
in user-overridable methods like get_app.

Add futures to travis and vm tests, update assorted dependency versions.

Cleanly shut down the ThreadPoolExecutor in Resolver tests.

This was somehow causing a problem on Ubuntu 10.04.

Doc coverage updates

Release notes for futures branch merge.

Merge branch 'futures'

Drop twisted tests from pypy-full configuration.

These tests seem to fail much more often on the futures branch than
master for some unknown reason. (the failures are in the
subprocess-related tests)

Release notes for stack_context changes

Merge branch 'master' into futures

Conflicts:
tornado/autoreload.py
tornado/ioloop.py
tornado/netutil.py
tornado/simple_httpclient.py
tornado/test/ioloop_test.py
tornado/test/runtests.py
tox.ini

Simplify StackContext logic and behavior.

Previously, there was an optimization to avoid recreating contexts
that were already on the stack.  This optimization rarely mattered
(especially given the movement to run all callbacks through
IOLoop.add_callback), and sometimes caused surprising behavior as
other exception handlers could be inserted between stack contexts that
expected to be adjacent to each other or to the wrapped function.  Now
each wrapped function recreates its full stack of contexts, even if
some of those contexts were already on the stack.

This change allows a use of NullContext in testing.py to be
removed.

Improve isolation of stack contexts.

Previously, a chain of callbacks started without any contexts could
accidentally pick up other contexts based on where the callbacks
were run.  This was due to a mistaken optimization in the no-context
case; now the behavior is the same whether the initial context was empty
or not.

Release note updates

Add IOLoop.add_callback_from_signal, which avoids deadlocks
which can occur when calling add_callback from a signal handler.

Also fix an issue in the recent set_wakeup_fd change when running
an IOLoop on a non-main thread.

Python 3.3 includes a fast time-independent comparison function, so use
it when available.

Apply my own suggested changes from pull request #555.

Allow non-standard HTTP codes to be passed with HTTPError.

Merge branch 'master' into merge

Conflicts:
tornado/simple_httpclient.py
tornado/test/web_test.py

Fix status/reason tests for py3, -curl, and -opt modes

Set Date header by default on all responses.

Support sending ping frames, trigger callback on pong frames.

Don't try to call set_wakeup_fd on windows, it crashes the process.

Use signal.set_wakeup_fd in IOLoop to close a race with signal handlers.

Subprocess.returncode should match subprocess.Popen.returncode.

Skip pipe and subprocess tests on windows.

Add a SIGCHILD handler to Subprocess.