- tor-commits - lists.torproject.org

[obfsproxy/master] Fixed various stuff pointed out by nickm.
by nickm＠torproject.org 30 May '11

30 May '11

commit 61a69edf357aea47c1e4f8c77c61b5bf6a53204f Author: George Kadianakis <desnacked(a)gmail.com> Date: Mon May 30 04:17:55 2011 +0200 Fixed various stuff pointed out by nickm. * Renamed RECV_OBFS2_PENDING to RECV_SEND_PENDING. * Removed ugly casts. * Added correct documentation for obfs2_recv(). --- src/network.c | 2 +- src/network.h | 4 ++-- src/protocols/obfs2.c | 28 +++++++++++++++------------- 3 files changed, 18 insertions(+), 16 deletions(-) diff --git a/src/network.c b/src/network.c index f629df9..3d3f172 100644 --- a/src/network.c +++ b/src/network.c @@ -318,7 +318,7 @@ obfuscated_read_cb(struct bufferevent *bev, void *arg) if (r == RECV_BAD) conn_free(conn); - else if (r == RECV_OBFS2_PENDING) + else if (r == RECV_SEND_PENDING) proto_send(conn->proto, bufferevent_get_input(conn->input), bufferevent_get_output(conn->output)); diff --git a/src/network.h b/src/network.h index d33b377..80b3055 100644 --- a/src/network.h +++ b/src/network.h @@ -29,11 +29,11 @@ enum recv_ret { RECV_INCOMPLETE, /* Originally needed by the obfs2 protocol but it might get other - users in the future. Maybe it should be renamed to something neutral. + users in the future. It means: "We have pending data that we have to send. You should do that by calling proto_send() immediately." */ - RECV_OBFS2_PENDING + RECV_SEND_PENDING }; typedef struct listener_t listener_t; diff --git a/src/protocols/obfs2.c b/src/protocols/obfs2.c index 975458a..c93db0e 100644 --- a/src/protocols/obfs2.c +++ b/src/protocols/obfs2.c @@ -355,10 +355,16 @@ init_crypto(void *s) } /* Called when we receive data in an evbuffer 'source': deobfuscates that data - * and writes it to 'dest'. + * and writes it to 'dest', by using protocol state 's' to get crypto keys. * - * Returns x for "don't call again till you have x bytes". 0 for "all ok". -1 - * for "fail, close" */ + * It returns: + * RECV_GOOD to say that everything went fine. + * RECV_BAD to say that something went bad. + * RECV_INCOMPLETE to say that we need more data to form an opinion. + * RECV_SEND_PENDING to say that everything went fine and on top of + * that we also have pending data that we have to send. This notifies + * our callers that they must call obfs2_send() immediately. + */ static enum recv_ret obfs2_recv(void *s, struct evbuffer *source, struct evbuffer *dest) @@ -373,7 +379,7 @@ obfs2_recv(void *s, struct evbuffer *source, uint32_t magic, plength; if (evbuffer_get_length(source) < OBFUSCATE_SEED_LENGTH+8) { /* data not here yet */ - return (enum recv_ret) RECV_INCOMPLETE; + return RECV_INCOMPLETE; } evbuffer_remove(source, buf, OBFUSCATE_SEED_LENGTH+8); @@ -386,7 +392,7 @@ obfs2_recv(void *s, struct evbuffer *source, /* Now we can set up all the keys from the seed */ if (init_crypto(state) < 0) - return (enum recv_ret) RECV_BAD; + return RECV_BAD; /* Decrypt the next 8 bytes */ stream_crypt(state->recv_padding_crypto, buf+OBFUSCATE_SEED_LENGTH, 8); @@ -396,13 +402,9 @@ obfs2_recv(void *s, struct evbuffer *source, magic = ntohl(magic); plength = ntohl(plength); if (magic != OBFUSCATE_MAGIC_VALUE) - return (enum recv_ret) RECV_BAD; + return RECV_BAD; if (plength > OBFUSCATE_MAX_PADDING) - return (enum recv_ret) RECV_BAD; - - /* XXX FIXME we should now be sending any 'pending_data_to_send' - but we can't send them from here, so we send them with - obfs2_send() when we next have to send data */ + return RECV_BAD; /* Now we're waiting for plength bytes of padding */ state->padding_left_to_read = plength; @@ -417,7 +419,7 @@ obfs2_recv(void *s, struct evbuffer *source, appropriately so that we call proto_send() right after we get out of here! */ if (state->pending_data_to_send) - r = RECV_OBFS2_PENDING; + r = RECV_SEND_PENDING; /* If we're still looking for padding, start pulling off bytes and discarding them. */ @@ -441,7 +443,7 @@ obfs2_recv(void *s, struct evbuffer *source, (int) evbuffer_get_length(source))); crypt_and_transmit(state->recv_crypto, source, dest); - if (r != RECV_OBFS2_PENDING) + if (r != RECV_SEND_PENDING) r = RECV_GOOD; return r;

1 0

[obfsproxy/master] Kill some more redundant casts
by nickm＠torproject.org 30 May '11

30 May '11

commit 1ee9c960c8fd5554369daa9f6606a875e38eda23 Author: Nick Mathewson <nickm(a)torproject.org> Date: Mon May 30 10:43:20 2011 -0400 Kill some more redundant casts --- src/protocols/dummy.c | 4 ++-- src/test/unittest_obfs2.c | 24 ++++++++++++------------ 2 files changed, 14 insertions(+), 14 deletions(-) diff --git a/src/protocols/dummy.c b/src/protocols/dummy.c index ed8d672..68bd7a1 100644 --- a/src/protocols/dummy.c +++ b/src/protocols/dummy.c @@ -60,7 +60,7 @@ dummy_recv(void *nothing, (void)nothing; if (evbuffer_add_buffer(dest,source)<0) - return (enum recv_ret) RECV_BAD; + return RECV_BAD; else - return (enum recv_ret) RECV_GOOD; + return RECV_GOOD; } diff --git a/src/test/unittest_obfs2.c b/src/test/unittest_obfs2.c index 9226cdf..b04040d 100644 --- a/src/test/unittest_obfs2.c +++ b/src/test/unittest_obfs2.c @@ -101,13 +101,13 @@ test_proto_handshake(void *data) /* We simulate the server receiving and processing the client's handshake message, by using proto_recv() on the output_buffer */ - tt_assert((enum recv_ret) RECV_GOOD == proto_recv(server_proto, output_buffer, dummy_buffer)); + tt_assert(RECV_GOOD == proto_recv(server_proto, output_buffer, dummy_buffer)); /* Now, we create the server's handshake and pass it to output_buffer */ tt_int_op(0, <=, proto_handshake(server_proto, output_buffer)); /* We simulate the client receiving and processing the server's handshake */ - tt_assert((enum recv_ret) RECV_GOOD == proto_recv(client_proto, output_buffer, dummy_buffer)); + tt_assert(RECV_GOOD == proto_recv(client_proto, output_buffer, dummy_buffer)); /* The handshake is now complete. We should have: client's send_crypto == server's recv_crypto @@ -170,10 +170,10 @@ test_proto_transfer(void *data) /* Handshake */ tt_int_op(0, <=, proto_handshake(client_proto, output_buffer)); - tt_assert((enum recv_ret) RECV_GOOD == proto_recv(server_proto, + tt_assert(RECV_GOOD == proto_recv(server_proto, output_buffer, dummy_buffer)); tt_int_op(0, <=, proto_handshake(server_proto, output_buffer)); - tt_assert((enum recv_ret) RECV_GOOD == proto_recv(client_proto, + tt_assert(RECV_GOOD == proto_recv(client_proto, output_buffer, dummy_buffer)); /* End of Handshake */ @@ -185,7 +185,7 @@ test_proto_transfer(void *data) evbuffer_add(dummy_buffer, msg1, 54); proto_send(client_proto, dummy_buffer, output_buffer); - tt_assert((enum recv_ret) RECV_GOOD == proto_recv(server_proto, + tt_assert(RECV_GOOD == proto_recv(server_proto, output_buffer, dummy_buffer)); n = evbuffer_peek(dummy_buffer, -1, NULL, &v[0], 2); @@ -201,7 +201,7 @@ test_proto_transfer(void *data) evbuffer_add(dummy_buffer, msg2, 55); tt_int_op(0, <=, proto_send(server_proto, dummy_buffer, output_buffer)); - tt_assert((enum recv_ret) RECV_GOOD == proto_recv(client_proto, + tt_assert(RECV_GOOD == proto_recv(client_proto, output_buffer, dummy_buffer)); n = evbuffer_peek(dummy_buffer, -1, NULL, &v[1], 2); @@ -298,7 +298,7 @@ test_proto_splitted_handshake(void *data) evbuffer_add(output_buffer, msgclient_1, OBFUSCATE_SEED_LENGTH+8+plength1_msg1); /* Server receives handshake part 1 */ - tt_assert((enum recv_ret) RECV_INCOMPLETE == proto_recv(server_proto, + tt_assert(RECV_INCOMPLETE == proto_recv(server_proto, output_buffer, dummy_buffer)); tt_assert(server_state->state == ST_WAIT_FOR_PADDING); @@ -311,7 +311,7 @@ test_proto_splitted_handshake(void *data) evbuffer_add(output_buffer, msgclient_2, plength1_msg2); /* Server receives handshake part 2 */ - tt_assert((enum recv_ret) RECV_GOOD == proto_recv(server_proto, + tt_assert(RECV_GOOD == proto_recv(server_proto, output_buffer, dummy_buffer)); tt_assert(server_state->state == ST_OPEN); @@ -341,7 +341,7 @@ test_proto_splitted_handshake(void *data) evbuffer_add(output_buffer, msgserver_1, OBFUSCATE_SEED_LENGTH+8); /* Client receives handshake part 1 */ - tt_assert((enum recv_ret) RECV_INCOMPLETE == proto_recv(client_proto, + tt_assert(RECV_INCOMPLETE == proto_recv(client_proto, output_buffer, dummy_buffer)); tt_assert(client_state->state == ST_WAIT_FOR_PADDING); @@ -354,7 +354,7 @@ test_proto_splitted_handshake(void *data) evbuffer_add(output_buffer, msgserver_2, plength2); /* Client receives handshake part 2 */ - tt_assert((enum recv_ret) RECV_GOOD == proto_recv(client_proto, + tt_assert(RECV_GOOD == proto_recv(client_proto, output_buffer, dummy_buffer)); tt_assert(client_state->state == ST_OPEN); @@ -446,7 +446,7 @@ test_proto_wrong_handshake_magic(void *data) evbuffer_add(output_buffer, msg, OBFUSCATE_SEED_LENGTH+8+plength); - tt_assert((enum recv_ret) RECV_BAD == proto_recv(server_proto, + tt_assert(RECV_BAD == proto_recv(server_proto, output_buffer, dummy_buffer)); tt_assert(server_state->state == ST_WAIT_FOR_KEY); @@ -526,7 +526,7 @@ test_proto_wrong_handshake_plength(void *data) evbuffer_add(output_buffer, msg, OBFUSCATE_SEED_LENGTH+8+plength); - tt_assert((enum recv_ret) RECV_BAD == proto_recv(server_proto, + tt_assert(RECV_BAD == proto_recv(server_proto, output_buffer, dummy_buffer)); tt_assert(server_state->state == ST_WAIT_FOR_KEY);

1 0

r24794: {website} remove the 'please consider a donation' sidenav link. (website/trunk/download/en)
by Andrew Lewman 30 May '11

30 May '11

Author: phobos Date: 2011-05-30 12:13:48 +0000 (Mon, 30 May 2011) New Revision: 24794 Modified: website/trunk/download/en/download-easy.wml website/trunk/download/en/download.wml Log: remove the 'please consider a donation' sidenav link. Modified: website/trunk/download/en/download-easy.wml =================================================================== --- website/trunk/download/en/download-easy.wml 2011-05-28 07:21:47 UTC (rev 24793) +++ website/trunk/download/en/download-easy.wml 2011-05-30 12:13:48 UTC (rev 24794) @@ -165,8 +165,6 @@ <li class="dropdown"><a href="<page download/download>#linux">Linux/Unix</a></li> <li class="dropdown"><a href="<page download/download>#smartphones">Smartphones</a></li> <li class="dropdown"><a href="<page download/download>#source">Source Code</a></li> -<li class="dropdown"><a href="<page donate/donate>">Please consider a -donation</a></li> </ul> </div> </div> Modified: website/trunk/download/en/download.wml =================================================================== --- website/trunk/download/en/download.wml 2011-05-28 07:21:47 UTC (rev 24793) +++ website/trunk/download/en/download.wml 2011-05-30 12:13:48 UTC (rev 24794) @@ -362,8 +362,6 @@ <li class="dropdown"><a href="#linux">Linux/Unix</a></li> <li class="dropdown"><a href="#smartphones">Smartphones</a></li> <li class="dropdown"><a href="#source">Source Code</a></li> -<li class="dropdown"><a href="<page donate/donate>">Please consider a -donation</a></li> </ul> </div> </div>

1 0

[metrics-web/master] Don't smush the labels together!
by karsten＠torproject.org 30 May '11

30 May '11

commit cb1b41de3257c7289c695d2f2b41c77c46b8cfbe Author: Karsten Loesing <karsten.loesing(a)gmx.net> Date: Mon May 30 09:36:46 2011 +0200 Don't smush the labels together! --- rserve/graphs.R | 176 +++++++++++++++++++++++++++++++----------------------- 1 files changed, 101 insertions(+), 75 deletions(-) diff --git a/rserve/graphs.R b/rserve/graphs.R index f268a34..cff39a2 100644 --- a/rserve/graphs.R +++ b/rserve/graphs.R @@ -254,6 +254,17 @@ countryname <- function(country) { res } +date_breaks <- function(days) { + length <- cut(days, c(0, 7, 12, 56, 180, 600, 5000, Inf), labels=FALSE) + major <- c("days", "2 days", "weeks", "months", "3 months", "years", + "5 years")[length] + minor <- c("10 years", "days", "days", "weeks", "months", "months", + "years")[length] + format <- c("%d-%b", "%d-%b", "%d-%b", "%b-%Y", "%b-%Y", "%Y", + "%Y")[length] + list(major = major, minor = minor, format = format) +} + plot_networksize <- function(start, end, path, dpi) { drv <- dbDriver("PostgreSQL") con <- dbConnect(drv, user = dbuser, password = dbpassword, dbname = db) @@ -283,14 +294,15 @@ plot_networksize <- function(start, end, path, dpi) { relays <- melt(relays, id = "date") bridges <- melt(bridges, id = "date") networksize <- rbind(relays, bridges) + date_breaks <- date_breaks( + as.numeric(max(as.Date(networksize$date, "%Y-%m-%d")) - + min(as.Date(networksize$date, "%Y-%m-%d")))) ggplot(networksize, aes(x = as.Date(date, "%Y-%m-%d"), y = value, colour = variable)) + geom_line(size = 1) + scale_x_date(name = paste("\nThe Tor Project - ", - "https://metrics.torproject.org/", sep = ""), format = - c("%d-%b", "%d-%b", "%b-%Y", "%b-%Y", "%Y", "%Y")[ - cut(as.numeric(max(as.Date(networksize$date, "%Y-%m-%d")) - - min(as.Date(networksize$date, "%Y-%m-%d"))), - c(0, 10, 56, 365, 730, 5000, Inf), labels=FALSE)]) + + "https://metrics.torproject.org/", sep = ""), + format = date_breaks$format, major = date_breaks$major, + minor = date_breaks$minor) + scale_y_continuous(name = "", limits = c(0, max(networksize$value, na.rm = TRUE))) + scale_colour_hue("", breaks = c("relays", "bridges"), @@ -329,14 +341,15 @@ plot_relaycountries <- function(start, end, country, path, dpi) { "Number of relays in all countries\n", paste("Number of relays in ", countryname(country), "\n", sep = "")) formatter <- function(x, ...) { format(x, scientific = FALSE, ...) } + date_breaks <- date_breaks( + as.numeric(max(as.Date(u$date, "%Y-%m-%d")) - + min(as.Date(u$date, "%Y-%m-%d")))) ggplot(u, aes(x = as.Date(date, "%Y-%m-%d"), y = relays)) + geom_line(size = 1) + scale_x_date(name = paste("\nThe Tor Project - ", - "https://metrics.torproject.org/", sep = ""), format = - c("%d-%b", "%d-%b", "%b-%Y", "%b-%Y", "%Y", "%Y")[ - cut(as.numeric(max(as.Date(u$date, "%Y-%m-%d")) - - min(as.Date(u$date, "%Y-%m-%d"))), - c(0, 10, 56, 365, 730, 5000, Inf), labels=FALSE)]) + + "https://metrics.torproject.org/", sep = ""), + format = date_breaks$format, major = date_breaks$major, + minor = date_breaks$minor) + scale_y_continuous(name = "", limits = c(0, max(u$relays, na.rm = TRUE)), formatter = formatter) + opts(title = title) @@ -360,15 +373,16 @@ plot_versions <- function(start, end, path, dpi) { date = as.Date(rep(end, 7)), version = known_versions, relays = rep(NA, 7)), versions) + date_breaks <- date_breaks( + as.numeric(max(as.Date(versions$date, "%Y-%m-%d")) - + min(as.Date(versions$date, "%Y-%m-%d")))) ggplot(versions, aes(x = as.Date(date, "%Y-%m-%d"), y = relays, colour = version)) + geom_line(size = 1) + scale_x_date(name = paste("\nThe Tor Project - ", - "https://metrics.torproject.org/", sep = ""), format = - c("%d-%b", "%d-%b", "%b-%Y", "%b-%Y", "%Y", "%Y")[ - cut(as.numeric(max(as.Date(versions$date, "%Y-%m-%d")) - - min(as.Date(versions$date, "%Y-%m-%d"))), - c(0, 10, 56, 365, 730, 5000, Inf), labels=FALSE)]) + + "https://metrics.torproject.org/", sep = ""), + format = date_breaks$format, major = date_breaks$major, + minor = date_breaks$minor) + scale_y_continuous(name = "", limits = c(0, max(versions$relays, na.rm = TRUE))) + scale_colour_hue(name = "Tor version", h.start = 280, @@ -388,15 +402,16 @@ plot_platforms <- function(start, end, path, dpi) { dbDisconnect(con) dbUnloadDriver(drv) platforms <- melt(platforms, id = "date") + date_breaks <- date_breaks( + as.numeric(max(as.Date(platforms$date, "%Y-%m-%d")) - + min(as.Date(platforms$date, "%Y-%m-%d")))) ggplot(platforms, aes(x = as.Date(date, "%Y-%m-%d"), y = value, colour = variable)) + geom_line(size = 1) + scale_x_date(name = paste("\nThe Tor Project - ", - "https://metrics.torproject.org/", sep = ""), format = - c("%d-%b", "%d-%b", "%b-%Y", "%b-%Y", "%Y", "%Y")[ - cut(as.numeric(max(as.Date(platforms$date, "%Y-%m-%d")) - - min(as.Date(platforms$date, "%Y-%m-%d"))), - c(0, 10, 56, 365, 730, 5000, Inf), labels=FALSE)]) + + "https://metrics.torproject.org/", sep = ""), + format = date_breaks$format, major = date_breaks$major, + minor = date_breaks$minor) + scale_y_continuous(name = "", limits = c(0, max(platforms$value, na.rm = TRUE))) + scale_colour_hue(name = "Platform", h.start = 180, @@ -425,15 +440,16 @@ plot_bandwidth <- function(start, end, path, dpi) { value = bw_desc$bwadvertised, variable = "bwadv"), data.frame(date = bw_hist$date, value = (bw_hist$read + bw_hist$written) / (2 * 86400), variable = "bwhist")) + date_breaks <- date_breaks( + as.numeric(max(as.Date(bandwidth$date, "%Y-%m-%d")) - + min(as.Date(bandwidth$date, "%Y-%m-%d")))) ggplot(bandwidth, aes(x = as.Date(date, "%Y-%m-%d"), y = value / 2^20, colour = variable)) + geom_line(size = 1) + scale_x_date(name = paste("\nThe Tor Project - ", - "https://metrics.torproject.org/", sep = ""), format = - c("%d-%b", "%d-%b", "%b-%Y", "%b-%Y", "%Y", "%Y")[ - cut(as.numeric(max(as.Date(bandwidth$date, "%Y-%m-%d")) - - min(as.Date(bandwidth$date, "%Y-%m-%d"))), - c(0, 10, 56, 365, 730, 5000, Inf), labels=FALSE)]) + + "https://metrics.torproject.org/", sep = ""), + format = date_breaks$format, major = date_breaks$major, + minor = date_breaks$minor) + scale_y_continuous(name="Bandwidth (MiB/s)", limits = c(0, max(bandwidth$value, na.rm = TRUE) / 2^20)) + scale_colour_hue(name = "", h.start = 90, @@ -470,15 +486,16 @@ plot_bwhist_flags <- function(start, end, path, dpi) { ifelse(bw$isguard, "Guard & Exit", "Exit only"), ifelse(bw$isguard, "Guard only", "Middle only")), value = (bw$read + bw$written) / 2) + date_breaks <- date_breaks( + as.numeric(max(as.Date(bw$date, "%Y-%m-%d")) - + min(as.Date(bw$date, "%Y-%m-%d")))) ggplot(bw, aes(x = as.Date(date, "%Y-%m-%d"), y = value / 2^20 / 86400, colour = variable)) + geom_line(size = 1) + scale_x_date(name = paste("\nThe Tor Project - ", - "https://metrics.torproject.org/", sep = ""), format = - c("%d-%b", "%d-%b", "%b-%Y", "%b-%Y", "%Y", "%Y")[ - cut(as.numeric(max(as.Date(bw$date, "%Y-%m-%d")) - - min(as.Date(bw$date, "%Y-%m-%d"))), - c(0, 10, 56, 365, 730, 5000, Inf), labels=FALSE)]) + + "https://metrics.torproject.org/", sep = ""), + format = date_breaks$format, major = date_breaks$major, + minor = date_breaks$minor) + scale_y_continuous(name="Bandwidth (MiB/s)", limits = c(0, max(bw$value, na.rm = TRUE) / 2^20 / 86400)) + scale_colour_hue(name = "") + @@ -502,15 +519,16 @@ plot_dirbytes <- function(start, end, path, dpi) { dirwrite = floor(dir$dw * dir$bwp / dir$bwd / 86400)) dir <- na.omit(dir) dir <- melt(dir, id = "date") + date_breaks <- date_breaks( + as.numeric(max(as.Date(dir$date, "%Y-%m-%d")) - + min(as.Date(dir$date, "%Y-%m-%d")))) ggplot(dir, aes(x = as.Date(date, "%Y-%m-%d"), y = value / 2^20, colour = variable)) + geom_line(size = 1) + scale_x_date(name = paste("\nThe Tor Project - ", - "https://metrics.torproject.org/", sep = ""), format = - c("%d-%b", "%d-%b", "%b-%Y", "%b-%Y", "%Y", "%Y")[ - cut(as.numeric(max(as.Date(dir$date, "%Y-%m-%d")) - - min(as.Date(dir$date, "%Y-%m-%d"))), - c(0, 10, 56, 365, 730, 5000, Inf), labels=FALSE)]) + + "https://metrics.torproject.org/", sep = ""), + format = date_breaks$format, major = date_breaks$major, + minor = date_breaks$minor) + scale_y_continuous(name="Bandwidth (MiB/s)", limits = c(0, max(dir$value, na.rm = TRUE) / 2^20)) + scale_colour_hue(name = "", @@ -538,14 +556,15 @@ plot_relayflags <- function(start, end, flags, granularity, path, dpi) { variable = paste("avg_", c("running", "exit", "guard", "fast", "stable"), sep = ""), value = rep(NA, 5)), networksize) + date_breaks <- date_breaks( + as.numeric(max(as.Date(networksize$date, "%Y-%m-%d")) - + min(as.Date(networksize$date, "%Y-%m-%d")))) ggplot(networksize, aes(x = as.Date(date, "%Y-%m-%d"), y = value, colour = variable)) + geom_line(size = 1) + scale_x_date(name = paste("\nThe Tor Project - ", - "https://metrics.torproject.org/", sep = ""), format = - c("%d-%b", "%d-%b", "%b-%Y", "%b-%Y", "%Y", "%Y")[ - cut(as.numeric(max(as.Date(networksize$date, "%Y-%m-%d")) - - min(as.Date(networksize$date, "%Y-%m-%d"))), - c(0, 10, 56, 365, 730, 5000, Inf), labels=FALSE)]) + + "https://metrics.torproject.org/", sep = ""), + format = date_breaks$format, major = date_breaks$major, + minor = date_breaks$minor) + scale_y_continuous(name = "", limits = c(0, max(networksize$value, na.rm = TRUE))) + scale_colour_hue(name = "Relay flags", h.start = 280, @@ -608,14 +627,15 @@ plot_direct_users <- function(start, end, country, path, dpi) { paste("Directly connecting users from ", countryname(country), "\n", sep = "")) formatter <- function(x, ...) { format(x, scientific = FALSE, ...) } + date_breaks <- date_breaks( + as.numeric(max(as.Date(u$date, "%Y-%m-%d")) - + min(as.Date(u$date, "%Y-%m-%d")))) ggplot(u, aes(x = as.Date(date, "%Y-%m-%d"), y = users)) + geom_line(size = 1) + scale_x_date(name = paste("\nThe Tor Project - ", - "https://metrics.torproject.org/", sep = ""), format = - c("%d-%b", "%d-%b", "%b-%Y", "%b-%Y", "%Y", "%Y")[ - cut(as.numeric(max(as.Date(u$date, "%Y-%m-%d")) - - min(as.Date(u$date, "%Y-%m-%d"))), - c(0, 10, 56, 365, 730, 5000, Inf), labels=FALSE)]) + + "https://metrics.torproject.org/", sep = ""), + format = date_breaks$format, major = date_breaks$major, + minor = date_breaks$minor) + scale_y_continuous(name = "", limits = c(0, max(u$users, na.rm = TRUE)), formatter = formatter) + opts(title = title) @@ -645,14 +665,15 @@ plot_bridge_users <- function(start, end, country, path, dpi) { "Bridge users from all countries\n", paste("Bridge users from ", countryname(country), "\n", sep = "")) formatter <- function(x, ...) { format(x, scientific = FALSE, ...) } + date_breaks <- date_breaks( + as.numeric(max(as.Date(bridgeusers$date, "%Y-%m-%d")) - + min(as.Date(bridgeusers$date, "%Y-%m-%d")))) ggplot(bridgeusers, aes(x = as.Date(date, "%Y-%m-%d"), y = users)) + geom_line(size = 1) + scale_x_date(name = paste("\nThe Tor Project - ", - "https://metrics.torproject.org/", sep = ""), format = - c("%d-%b", "%d-%b", "%b-%Y", "%b-%Y", "%Y", "%Y")[ - cut(as.numeric(max(as.Date(bridgeusers$date, "%Y-%m-%d")) - - min(as.Date(bridgeusers$date, "%Y-%m-%d"))), - c(0, 10, 56, 365, 730, 5000, Inf), labels=FALSE)]) + + "https://metrics.torproject.org/", sep = ""), + format = date_breaks$format, major = date_breaks$major, + minor = date_breaks$minor) + scale_y_continuous(name = "", limits = c(0, max(bridgeusers$users, na.rm = TRUE)), formatter = formatter) + opts(title = title) @@ -682,14 +703,15 @@ plot_gettor <- function(start, end, bundle, path, dpi) { "Total packages requested from GetTor per day\n", paste("Tor Browser Bundles (", bundle, ") requested from GetTor per day\n", sep = "")) + date_breaks <- date_breaks( + as.numeric(max(as.Date(downloads$date, "%Y-%m-%d")) - + min(as.Date(downloads$date, "%Y-%m-%d")))) ggplot(downloads, aes(x = as.Date(date, "%Y-%m-%d"), y = downloads)) + geom_line(size = 1) + scale_x_date(name = paste("\nThe Tor Project - ", - "https://metrics.torproject.org/", sep = ""), format = - c("%d-%b", "%d-%b", "%b-%Y", "%b-%Y", "%Y", "%Y")[ - cut(as.numeric(max(as.Date(downloads$date, "%Y-%m-%d")) - - min(as.Date(downloads$date, "%Y-%m-%d"))), - c(0, 10, 56, 365, 730, 5000, Inf), labels=FALSE)]) + + "https://metrics.torproject.org/", sep = ""), + format = date_breaks$format, major = date_breaks$major, + minor = date_breaks$minor) + scale_y_continuous(name = "", limits = c(0, max(downloads$downloads, na.rm = TRUE))) + opts(title = title) @@ -721,17 +743,18 @@ plot_torperf <- function(start, end, source, filesize, path, dpi) { label = c("5 MiB", "1 MiB", "50 KiB"), stringsAsFactors = FALSE) filesizeStr <- filesizes[filesizes$filesize == filesize, "label"] maxY <- max(torperf$q3, na.rm = TRUE) + date_breaks <- date_breaks( + as.numeric(max(as.Date(torperf$date, "%Y-%m-%d")) - + min(as.Date(torperf$date, "%Y-%m-%d")))) ggplot(torperf, aes(x = as.Date(date, "%Y-%m-%d"), y = md/1e3, fill = "line")) + geom_line(colour = colour, size = 0.75) + geom_ribbon(data = torperf, aes(x = date, ymin = q1/1e3, ymax = q3/1e3, fill = "ribbon")) + scale_x_date(name = paste("\nThe Tor Project - ", - "https://metrics.torproject.org/", sep = ""), format = - c("%d-%b", "%d-%b", "%b-%Y", "%b-%Y", "%Y", "%Y")[ - cut(as.numeric(max(as.Date(torperf$date, "%Y-%m-%d")) - - min(as.Date(torperf$date, "%Y-%m-%d"))), - c(0, 10, 56, 365, 730, 5000, Inf), labels=FALSE)]) + + "https://metrics.torproject.org/", sep = ""), + format = date_breaks$format, major = date_breaks$major, + minor = date_breaks$minor) + scale_y_continuous(name = "", limits = c(0, maxY) / 1e3) + scale_fill_manual(name = paste("Measured times on", ifelse(source == "all", "all sources", source), "per day"), @@ -777,15 +800,16 @@ plot_torperf_failures <- function(start, end, source, filesize, path, value = ifelse(torperf$requests > 0, torperf$failures / torperf$requests, 0), variable = "failures")) + date_breaks <- date_breaks( + as.numeric(max(as.Date(torperf$date, "%Y-%m-%d")) - + min(as.Date(torperf$date, "%Y-%m-%d")))) ggplot(torperf, aes(x = as.Date(date, "%Y-%m-%d"), y = value, colour = variable)) + geom_point(size = 2) + scale_x_date(name = paste("\nThe Tor Project - ", - "https://metrics.torproject.org/", sep = ""), format = - c("%d-%b", "%d-%b", "%b-%Y", "%b-%Y", "%Y", "%Y")[ - cut(as.numeric(max(as.Date(torperf$date, "%Y-%m-%d")) - - min(as.Date(torperf$date, "%Y-%m-%d"))), - c(0, 10, 56, 365, 730, 5000, Inf), labels=FALSE)]) + + "https://metrics.torproject.org/", sep = ""), + format = date_breaks$format, major = date_breaks$major, + minor = date_breaks$minor) + scale_y_continuous(name = "", formatter = "percent") + scale_colour_hue(name = paste("Problems encountered on", ifelse(source == "all", "all sources", source)), @@ -809,15 +833,16 @@ plot_connbidirect <- function(start, end, path, dpi) { connbidirect <- data.frame(date = c$date, c[, 2:4] / (c$readnum + c$writenum + c$bothnum)) connbidirect <- melt(connbidirect, id = "date") + date_breaks <- date_breaks( + as.numeric(max(as.Date(connbidirect$date, "%Y-%m-%d")) - + min(as.Date(connbidirect$date, "%Y-%m-%d")))) ggplot(connbidirect, aes(x = as.Date(date, "%Y-%m-%d"), y = value, colour = variable)) + geom_point(size = 2.5) + scale_x_date(name = paste("\nThe Tor Project - ", - "https://metrics.torproject.org/", sep = ""), format = - c("%d-%b", "%d-%b", "%b-%Y", "%b-%Y", "%Y", "%Y")[ - cut(as.numeric(max(as.Date(connbidirect$date, "%Y-%m-%d")) - - min(as.Date(connbidirect$date, "%Y-%m-%d"))), - c(0, 10, 56, 365, 730, 5000, Inf), labels=FALSE)]) + + "https://metrics.torproject.org/", sep = ""), + format = date_breaks$format, major = date_breaks$major, + minor = date_breaks$minor) + scale_y_continuous(name = "", formatter = "percent") + scale_colour_hue("", breaks = c("readnum", "writenum", "bothnum"), labels = c("Mostly reading", "Mostly writing", @@ -844,14 +869,15 @@ plot_routerdetail <- function(fingerprint, path) { ## TODO We should add NA's for missing dates. dbDisconnect(con) dbUnloadDriver(drv) + date_breaks <- date_breaks( + as.numeric(max(as.Date(routerdetail$date, "%Y-%m-%d")) - + min(as.Date(routerdetail$date, "%Y-%m-%d")))) ggplot(routerdetail, aes(x = as.Date(date, "%Y-%m-%d"), y = bw)) + geom_line(size = 1) + scale_x_date(name = paste("\nThe Tor Project - ", - "https://metrics.torproject.org/", sep = ""), format = - c("%d-%b", "%d-%b", "%b-%Y", "%b-%Y", "%Y", "%Y")[ - cut(as.numeric(max(as.Date(routerdetail$date, "%Y-%m-%d")) - - min(as.Date(routerdetail$date, "%Y-%m-%d"))), - c(0, 10, 56, 365, 730, 5000, Inf), labels=FALSE)]) + + "https://metrics.torproject.org/", sep = ""), + format = date_breaks$format, major = date_breaks$major, + minor = date_breaks$minor) + scale_y_continuous(name = "") + opts(title = paste("Bandwidth history for ", fingerprint, "\n", sep = ""))

1 0

[tor/release-0.2.2] answer an XXX nickm asked in aa950e6c4
by arma＠torproject.org 30 May '11

30 May '11

commit 5f182ea10e25ed027cf75627f74013e7c6da2038 Author: Roger Dingledine <arma(a)torproject.org> Date: Sun May 29 18:52:37 2011 -0400 answer an XXX nickm asked in aa950e6c4 --- src/or/circuitbuild.c | 2 +- src/or/main.c | 5 ++++- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/src/or/circuitbuild.c b/src/or/circuitbuild.c index 108007e..9509b5a 100644 --- a/src/or/circuitbuild.c +++ b/src/or/circuitbuild.c @@ -262,7 +262,7 @@ circuit_build_times_test_frequency(void) } /** - * Retrieve and bounds-check the cbtmintimeout consensus paramter. + * Retrieve and bounds-check the cbtmintimeout consensus parameter. * * Effect: This is the minimum allowed timeout value in milliseconds. * The minimum is to prevent rounding to 0 (we only check once diff --git a/src/or/main.c b/src/or/main.c index d1ceeec..4fc6437 100644 --- a/src/or/main.c +++ b/src/or/main.c @@ -1162,7 +1162,10 @@ run_scheduled_events(time_t now) * it's not comfortable with the number of available circuits. */ /* XXXX022 If our circuit build timeout is much lower than a second, maybe - we should do this more often? */ + * we should do this more often? -NM + * It can't be lower than 1.5 seconds currently; see + * circuit_build_times_min_timeout(). -RD + */ circuit_expire_building(); /** 3b. Also look at pending streams and prune the ones that 'began'

1 0

[tor/release-0.2.2] Merge branch 'maint-0.2.2' into release-0.2.2
by arma＠torproject.org 30 May '11

30 May '11

commit 688400c596c09e9a6d1a26668e4843ccd946a7be Merge: 6746293 5f182ea Author: Roger Dingledine <arma(a)torproject.org> Date: Sun May 29 19:11:22 2011 -0400 Merge branch 'maint-0.2.2' into release-0.2.2 src/or/circuitbuild.c | 2 +- src/or/main.c | 5 ++++- 2 files changed, 5 insertions(+), 2 deletions(-)

1 0

[tor/master] Merge branch 'maint-0.2.2'
by arma＠torproject.org 30 May '11

30 May '11

commit 6917728637bdbaed86d48d394eab6a063c319269 Merge: e676db6 5f182ea Author: Roger Dingledine <arma(a)torproject.org> Date: Sun May 29 19:11:07 2011 -0400 Merge branch 'maint-0.2.2' src/or/circuitbuild.c | 2 +- src/or/main.c | 5 ++++- 2 files changed, 5 insertions(+), 2 deletions(-)

1 0

[tor/master] answer an XXX nickm asked in aa950e6c4
by arma＠torproject.org 30 May '11

30 May '11

commit 5f182ea10e25ed027cf75627f74013e7c6da2038 Author: Roger Dingledine <arma(a)torproject.org> Date: Sun May 29 18:52:37 2011 -0400 answer an XXX nickm asked in aa950e6c4 --- src/or/circuitbuild.c | 2 +- src/or/main.c | 5 ++++- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/src/or/circuitbuild.c b/src/or/circuitbuild.c index 108007e..9509b5a 100644 --- a/src/or/circuitbuild.c +++ b/src/or/circuitbuild.c @@ -262,7 +262,7 @@ circuit_build_times_test_frequency(void) } /** - * Retrieve and bounds-check the cbtmintimeout consensus paramter. + * Retrieve and bounds-check the cbtmintimeout consensus parameter. * * Effect: This is the minimum allowed timeout value in milliseconds. * The minimum is to prevent rounding to 0 (we only check once diff --git a/src/or/main.c b/src/or/main.c index d1ceeec..4fc6437 100644 --- a/src/or/main.c +++ b/src/or/main.c @@ -1162,7 +1162,10 @@ run_scheduled_events(time_t now) * it's not comfortable with the number of available circuits. */ /* XXXX022 If our circuit build timeout is much lower than a second, maybe - we should do this more often? */ + * we should do this more often? -NM + * It can't be lower than 1.5 seconds currently; see + * circuit_build_times_min_timeout(). -RD + */ circuit_expire_building(); /** 3b. Also look at pending streams and prune the ones that 'began'

1 0

[tor/maint-0.2.2] answer an XXX nickm asked in aa950e6c4
by arma＠torproject.org 30 May '11

30 May '11

commit 5f182ea10e25ed027cf75627f74013e7c6da2038 Author: Roger Dingledine <arma(a)torproject.org> Date: Sun May 29 18:52:37 2011 -0400 answer an XXX nickm asked in aa950e6c4 --- src/or/circuitbuild.c | 2 +- src/or/main.c | 5 ++++- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/src/or/circuitbuild.c b/src/or/circuitbuild.c index 108007e..9509b5a 100644 --- a/src/or/circuitbuild.c +++ b/src/or/circuitbuild.c @@ -262,7 +262,7 @@ circuit_build_times_test_frequency(void) } /** - * Retrieve and bounds-check the cbtmintimeout consensus paramter. + * Retrieve and bounds-check the cbtmintimeout consensus parameter. * * Effect: This is the minimum allowed timeout value in milliseconds. * The minimum is to prevent rounding to 0 (we only check once diff --git a/src/or/main.c b/src/or/main.c index d1ceeec..4fc6437 100644 --- a/src/or/main.c +++ b/src/or/main.c @@ -1162,7 +1162,10 @@ run_scheduled_events(time_t now) * it's not comfortable with the number of available circuits. */ /* XXXX022 If our circuit build timeout is much lower than a second, maybe - we should do this more often? */ + * we should do this more often? -NM + * It can't be lower than 1.5 seconds currently; see + * circuit_build_times_min_timeout(). -RD + */ circuit_expire_building(); /** 3b. Also look at pending streams and prune the ones that 'began'

1 0

[tor/release-0.2.2] fold in recent changes entries
by arma＠torproject.org 30 May '11

30 May '11

commit 67462937a24d24c04146e2547d853b4a42c0d718 Author: Roger Dingledine <arma(a)torproject.org> Date: Sun May 29 15:34:38 2011 -0400 fold in recent changes entries --- ChangeLog | 22 +++++++++++++++++++--- changes/bug2574 | 3 --- changes/bug3208 | 6 ------ changes/exit-policy-default-is-not-a-prefix | 5 ----- changes/feature3049 | 6 ------ 5 files changed, 19 insertions(+), 23 deletions(-) diff --git a/ChangeLog b/ChangeLog index 92478b3..42d4e14 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,14 +1,30 @@ -Changes in version 0.2.2.28-beta - 2011-05-2? +Changes in version 0.2.2.28-beta - 2011-05-3? o Major bugfixes: - Don't decide to make a new descriptor when receiving a HUP signal. This bug has caused a lot of 0.2.2.x relays to disappear from the consensus periodically. Fixes the most common case of triggering bug 1810; bugfix on 0.2.2.7-alpha. + - Actually allow nameservers with IPv6 addresses. Fixes bug 2574. + + o Major features: + - Add an __OwningControllerProcess configuration option and a + TAKEOWNERSHIP control-port command. Now a Tor controller can ensure + that when it exits, Tor will shut down. Implements feature 3049. + + o Minor bugfixes: + - Fix warnings from GCC 4.6's "-Wunused-but-set-variable" option. + - Remove a trailing asterisk from "exit-policy/default" in the + output of the control port command "GETINFO info/names". Bugfix + on 0.1.2.5-alpha. o Minor features: - Relays now log the reason for publishing a new relay descriptor, - so we have a better chance of hunting down the root cause of bug - 1810. Resolves ticket 3252. + so we have a better chance of hunting down instances of bug 1810. + Resolves ticket 3252. + + o Removed options: + - Remove undocumented option "-F" from tor-resolve: it hasn't done + anything since 0.2.1.16-rc. Changes in version 0.2.2.27-beta - 2011-05-18 diff --git a/changes/bug2574 b/changes/bug2574 deleted file mode 100644 index 5cf2dae..0000000 --- a/changes/bug2574 +++ /dev/null @@ -1,3 +0,0 @@ - o Minor features: - - Allow nameservers with IPv6 address. Fixes bug 2574. - diff --git a/changes/bug3208 b/changes/bug3208 deleted file mode 100644 index fd737ba..0000000 --- a/changes/bug3208 +++ /dev/null @@ -1,6 +0,0 @@ - o Removed options: - - Remove undocumented option "-F" from tor-resolve: it hasn't done - anything since 0.2.1.16-rc. - - o Minor bugfixes: - - Fix warnings from GCC 4.6's "-Wunused-but-set-variable" option. diff --git a/changes/exit-policy-default-is-not-a-prefix b/changes/exit-policy-default-is-not-a-prefix deleted file mode 100644 index 6eb1e8d..0000000 --- a/changes/exit-policy-default-is-not-a-prefix +++ /dev/null @@ -1,5 +0,0 @@ - o Minor bugfixes: - - Remove a trailing asterisk from "exit-policy/default" in the - output of the control port command "GETINFO info/names". Bugfix - on 0.1.2.5-alpha. - diff --git a/changes/feature3049 b/changes/feature3049 deleted file mode 100644 index 7960a1f..0000000 --- a/changes/feature3049 +++ /dev/null @@ -1,6 +0,0 @@ - o Major features: - - Add an __OwningControllerProcess configuration option and a - TAKEOWNERSHIP control-port command, so that a Tor controller can - ensure that when it exits, Tor will shut down. Implements - feature 3049. -

1 0