Fix issue with truncating recorded ip addresses

Shun Fan · Shun Fan · commit 6e2cc569f068 · 2016-02-05T16:47:48.000-05:00
diff --git a/managed_vms/cloudsql/src/main/java/com/example/managedvms/cloudsql/CloudSqlServlet.java b/managed_vms/cloudsql/src/main/java/com/example/managedvms/cloudsql/CloudSqlServlet.java
@@ -47,9 +47,11 @@ public void doGet(HttpServletRequest req, HttpServletResponse resp) throws IOExc
     String userIp = req.getRemoteAddr();
     InetAddress address = InetAddress.getByName(userIp);
     if (address instanceof Inet6Address) {
-      userIp = userIp.substring(0, userIp.indexOf(":", 2)) + ":*:*:*:*:*:*";
+      // nest indexOf calls to find the second occurrence of a character in a string
+      // an alternative is to use Apache Commons Lang: StringUtils.ordinalIndexOf()
+      userIp = userIp.substring(0, userIp.indexOf(":", userIp.indexOf(":") + 1)) + ":*:*:*:*:*:*";
     } else if (address instanceof Inet4Address) {
-      userIp = userIp.substring(0, userIp.indexOf(".", 2)) + ".*.*";
+      userIp = userIp.substring(0, userIp.indexOf(".", userIp.indexOf(".") + 1)) + ".*.*";
     }
 
     final String createTableSql = "CREATE TABLE IF NOT EXISTS visits ( visit_id INT NOT NULL "
