r11973@Kushana: nickm | 2007-01-15 16:43:25 -0500

 Answer questions from r9348.


svn:r9358

1 files changed, 4 insertions, 3 deletions

diff --git a/src/or/dns.c b/src/or/dns.c
index a37e35550a..c378ad88b1 100644
--- a/src/or/dns.c
+++ b/src/or/dns.c
@@ -193,9 +193,7 @@ evdns_log_cb(int warn, const char *msg)
   } else if (!strcmpstart(msg, "Nameserver ") &&
              (cp=strstr(msg, " is back up"))) {
     char *ns = tor_strndup(msg+11, cp-(msg+11));
-    /* XXX012 we should set warn=0 if some nameservers were
-     * already up -- so we only shout when we actually change
-     * a meaningful state. */
+    warn = 0; /* It's never a warning when a nameserver comes back up. */
     control_event_server_status(LOG_NOTICE,
                                 "NAMESERVER_STATUS NS=%s STATUS=UP", ns);
     tor_free(ns);
@@ -1692,6 +1690,9 @@ launch_resolve(edge_connection_t *exitconn, or_circuit_t *circ)
     /* XXX012 the above warning triggers sometimes on my exit
      * node, with "error 1". Is there anything the user can do to
      * resolve this? -RD */
+    /* XXX012 Are you sure I didn't fix that in version r9336 with
+     * ServerDNSAllowNonRFC953Addreesses? If it's still not working,
+     * let me know which addresses are breaking. -NM */
     if (exitconn->_base.purpose == EXIT_PURPOSE_RESOLVE) {
       if (evdns_err_is_transient(r))
         send_resolved_cell(exitconn, circ, RESOLVED_TYPE_ERROR_TRANSIENT);