r17270@catbus: nickm | 2007-12-20 00:04:21 -0500

When we decide to send a 503 in response to a request for server descriptors, disable spooling so that we do not then send the descriptors anyway.  Fixes bug 539.


svn:r12882
This commit is contained in:
Nick Mathewson 2007-12-20 05:06:02 +00:00
parent 5c33c0632e
commit 6de2b484cf
2 changed files with 6 additions and 1 deletions

View File

@ -85,7 +85,11 @@ Changes in version 0.2.0.13-alpha - 2007-12-??
- Fix compilation with --disable-threads set.
- Authorities decide whether they're authoritative for a given router
based on the router's purpose.
- Don't crash on name lookup when we have no current consensus.
- Don't crash on name lookup when we have no current consensus. Fixes
bug 538; bugfix on 0.2.0.x.
- When we decide to send a 503 response to a request for servers, do
not then also send the server descriptors: this defeats the whole
purpose. Fixes bug 539; bugfix on 0.1.2.x.
o Minor features:
- On USR1, when dmalloc is in use, log the top 10 memory

View File

@ -2440,6 +2440,7 @@ directory_handle_command_get(dir_connection_t *conn, const char *headers,
"Client asked for server descriptors, but we've been "
"writing too many bytes lately. Sending 503 Dir busy.");
write_http_status_line(conn, 503, "Directory busy, try again later");
conn->dir_spool_src = DIR_SPOOL_NONE;
goto done;
}
write_http_response_header(conn, -1, deflated, cache_lifetime);