[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[or-cvs] r9158: minor cleanups (in tor/trunk: . doc src/or)



Author: arma
Date: 2006-12-20 04:25:58 -0500 (Wed, 20 Dec 2006)
New Revision: 9158

Modified:
   tor/trunk/ChangeLog
   tor/trunk/doc/TODO
   tor/trunk/src/or/connection_edge.c
   tor/trunk/src/or/or.h
Log:
minor cleanups


Modified: tor/trunk/ChangeLog
===================================================================
--- tor/trunk/ChangeLog	2006-12-19 19:49:03 UTC (rev 9157)
+++ tor/trunk/ChangeLog	2006-12-20 09:25:58 UTC (rev 9158)
@@ -23,12 +23,14 @@
       options files.
     - Reject *:563 (NTTPS) in the default exit policy. We already reject
       NNTP by default, so this seems like a sensible addition.
-    - Authorities do not recommend exits as guards if this would shift excess
-      load to the exit nodes.
+    - Authorities do not recommend exits as guards if this would shift
+      excess load to the exit nodes.
     - Avoid some inadvertent info leaks by making clients reject hostnames
-      with invalid characters.  Add an option to disable this behavior,
-      in case somebody is running a private network with hosts called @, !,
-      and #.
+      with invalid characters. Add an option "AllowNonRFC953Hostnames"
+      to disable this behavior, in case somebody is running a private
+      network with hosts called @, !, and #.
+    - Add a new address-spec.txt document to describe our special-case
+      addresses: .exit, .onion, and .noconnnect.
 
   o Security bugfixes:
     - Stop sending the HttpProxyAuthenticator string to directory
@@ -50,27 +52,23 @@
       after that. (May fix bug 326.)
 
   o Minor bugfixes:
-    - Fix a bug when a PF socket is first used.  (Patch from Fabian
-      Keil.)
+    - Fix a bug when a PF socket is first used. (Patch from Fabian Keil.)
     - Fix an assert failure when a directory authority sets
       AuthDirRejectUnlisted and then receives a descriptor from an
       unlisted router (reported by seeess).
     - Fix a bug on the Windows implementation of tor_mmap_file that
-      would prevent the cached-routers file from ever loading. (reported by
-      John Kimble.)
+      would prevent the cached-routers file from ever loading (reported
+      by John Kimble).
     - Fix a bug in 0.1.2.2-alpha that prevented clients from asking
       to resolve an address at a given exit node even when they ask for
       it by name.
     - Routers no longer ever list themselves in their "family" line,
       even if configured to do so.  This makes it easier to configure
       family lists efficiently.
-    - Remove an artificial (but quite high) restriction on expected
-      bandwidth, so that accounting won't break once we all have gigabit
-      connections to our homes.
-    - When running as a server, don't fall back to 127.0.0.1 when
-      no nameservers are configured in /etc/resolv.conf; instead, make
-      the user fix resolv.conf or specify nameservers explicitly. (Resolves
-      Bug 363.)
+    - When running as a server, don't fall back to 127.0.0.1 when no
+      nameservers are configured in /etc/resolv.conf; instead, make the
+      user fix resolv.conf or specify nameservers explicitly. (Resolves
+      bug 363.)
 
   o Controller features:
     - Have GETINFO dir/status/* work on hosts with DirPort disabled.

Modified: tor/trunk/doc/TODO
===================================================================
--- tor/trunk/doc/TODO	2006-12-19 19:49:03 UTC (rev 9157)
+++ tor/trunk/doc/TODO	2006-12-20 09:25:58 UTC (rev 9158)
@@ -59,12 +59,15 @@
         a new line in the status entry. "Tor 0.1.2.2-alpha". If it's
         a version, treat it like one. If it's something else, assume
         it's at least 0.1.2.x.
+        maybe we could have it be a new 'v' line in the status, with
+        key=value syntax. so we could have a 'tor' version, but we
+        could also have a 'conn' version, a 'dir' version, etc down
+        the road. and one day maybe the 'tor' key would be deprecated.
 
   o Document .noconnect addresses...
     A new file 'address-spec.txt' that describes .exit, .onion,
     .noconnect, etc?
 
-
   - Servers are easy to setup and run: being a relay is about as easy as
     being a client.
     . Reduce resource load
@@ -102,6 +105,7 @@
       o address_is_invalid_destination() is the right thing to call here
         (and feel free to make that function smarter)
       o add a config option to turn it off.
+      - and a man page for that option
     - Bug 364: notice when all the DNS requests we get back (including a few
       well-known sites) are all going to the same place.
     o Bug 363: Warn and die if we can't find a nameserver and we're running a

Modified: tor/trunk/src/or/connection_edge.c
===================================================================
--- tor/trunk/src/or/connection_edge.c	2006-12-19 19:49:03 UTC (rev 9157)
+++ tor/trunk/src/or/connection_edge.c	2006-12-20 09:25:58 UTC (rev 9158)
@@ -795,7 +795,7 @@
   struct in_addr in;
   /* <address>.<hex or nickname>.exit\0  or just  <address>\0 */
   char extendedaddress[MAX_SOCKS_ADDR_LEN+MAX_VERBOSE_NICKNAME_LEN+10];
-  /* 123.123.123.123.<hex or nickname>.exit\0   or just  123.123.123.123\0 */
+  /* 123.123.123.123.<hex or nickname>.exit\0  or just  123.123.123.123\0 */
   char extendedval[INET_NTOA_BUF_LEN+MAX_VERBOSE_NICKNAME_LEN+10];
   char valbuf[INET_NTOA_BUF_LEN];
 

Modified: tor/trunk/src/or/or.h
===================================================================
--- tor/trunk/src/or/or.h	2006-12-19 19:49:03 UTC (rev 9157)
+++ tor/trunk/src/or/or.h	2006-12-20 09:25:58 UTC (rev 9158)
@@ -1656,8 +1656,8 @@
                                * same network zone in the same circuit. */
   int TunnelDirConns; /**< If true, use BEGIN_DIR rather than BEGIN when
                        * possible. */
-  int AllowNonRFC953Hostnames;  /**< If true, we allow connections to hostnames
-                                 * with weird characters. */
+  int AllowNonRFC953Hostnames; /**< If true, we allow connections to hostnames
+                                * with weird characters. */
 } or_options_t;
 
 /** Persistent state for an onion router, as saved to disk. */