[tbb-commits] [torbutton/master] Bug 14939: Support IPv6 addresses in Tor Circuit Display

[richard at torproject.org]

commit bf12640cca3e758a565cb3f9f5a2ad8a5f22bb92
Author: Illia Volochii <illia.volochii at gmail.com>
Date:   Mon Nov 1 20:20:40 2021 +0200

    Bug 14939: Support IPv6 addresses in Tor Circuit Display
    
    Fix the visualization of bridge addresses when using IPv6, and display
    both IPv4 and IPv6 addresses on normal relays.
---
 chrome/content/tor-circuit-display.js | 39 ++++++++++++++++++++++-------------
 1 file changed, 25 insertions(+), 14 deletions(-)

diff --git a/chrome/content/tor-circuit-display.js b/chrome/content/tor-circuit-display.js
index d6034384..440e9de9 100644
--- a/chrome/content/tor-circuit-display.js
+++ b/chrome/content/tor-circuit-display.js
@@ -70,38 +70,49 @@ let getBridge = async function (controller, id) {
 };
 
 // nodeDataForID(controller, id)__.
-// Returns the type, IP and country code of a node with given ID.
+// Returns the type, IP addresses and country code of a node with given ID.
 // Example: `nodeDataForID(controller, "20BC91DC525C3DC9974B29FBEAB51230DE024C44")`
-// => `{ type : "default", ip : "12.23.34.45", countryCode : "fr" }`
+// => `{ type: "default", ipAddrs: ["12.23.34.45", "2001:db8::"], countryCode: "fr" }`
 let nodeDataForID = async function (controller, id) {
-  let result = {},
-      bridge = await getBridge(controller, id); // type, ip, countryCode;
+  let result = {ipAddrs: []};
+  const bridge = await getBridge(controller, id); // type, ip, countryCode;
+  const addrRe = /^\[?([^\]]+)\]?:\d+$/
   if (bridge) {
     result.type = "bridge";
     result.bridgeType = bridge.type;
     // Attempt to get an IP address from bridge address string.
     try {
-      result.ip = bridge.address.split(":")[0];
-    } catch (e) { }
+      const ip = bridge.address.match(addrRe)[1];
+      if (!ip.startsWith("0.")) {
+        result.ipAddrs = [ip];
+      }
+    } catch (e) {
+    }
   } else {
     // either dealing with a relay, or a bridge whose fingerprint is not saved in torrc
     try {
-      let statusMap = await controller.getInfo("ns/id/" + id);
+      const statusMap = await controller.getInfo("ns/id/" + id);
       result.type = "default";
-      result.ip = statusMap.IP;
+      if (!statusMap.IP.startsWith("0.")) {
+        result.ipAddrs.push(statusMap.IP);
+      }
+      try {
+        result.ipAddrs.push(statusMap.IPv6.match(addrRe)[1]);
+      } catch (e) {
+      }
     } catch (e) {
       // getInfo will throw if the given id is not a relay
       // this probably means we are dealing with a user-provided bridge with no fingerprint
       result.type = "bridge";
-      // we don't know the ip or type, so leave blank
-      result.ip = "";
+      // we don't know the ip/ipv6 or type, so leave blank
+      result.ipAddrs = [];
       result.bridgeType = "";
     }
   }
-  if (result.ip) {
+  if (result.ipAddrs.length) {
     // Get the country code for the node's IP address.
     try {
-      let countryCode = await controller.getInfo("ip-to-country/" + result.ip);
+      const countryCode = await controller.getInfo("ip-to-country/" + result.ipAddrs[0]);
       result.countryCode = countryCode === "??" ? null : countryCode;
     } catch (e) { }
   }
@@ -305,8 +316,8 @@ let updateCircuitDisplay = function () {
       } else if (nodeData[i].type == "default") {
         relayText = localizedCountryNameFromCode(nodeData[i].countryCode);
       }
-      let ip = nodeData[i].ip.startsWith("0.") ? "" : nodeData[i].ip;
-      li(relayText, " ", ["span", { class: "circuit-ip-address" }, ip], " ",
+      const ipAddrs = nodeData[i].ipAddrs.join(", ");
+      li(relayText, " ", ["span", { class: "circuit-ip-address" }, ipAddrs], " ",
          (i === 0 && nodeData[0].type !== "bridge") ?
            ["span", { class: "circuit-guard-info" }, uiString("guard")] : null);
     }