[tor-commits] [exonerator/master] Make changes suggested by Geoff Down and Jens Kubieziel.

[karsten at torproject.org]

commit e2685311489fdc46c941cef2a24cdcc4407e2ca3
Author: Karsten Loesing <karsten.loesing at gmx.net>
Date:   Wed Jul 15 08:16:06 2015 +0200

    Make changes suggested by Geoff Down and Jens Kubieziel.
---
 res/ExoneraTor_de.properties                         |    4 ++--
 src/org/torproject/exonerator/ExoneraTorServlet.java |   18 ++++++++++++------
 2 files changed, 14 insertions(+), 8 deletions(-)

diff --git a/res/ExoneraTor_de.properties b/res/ExoneraTor_de.properties
index ee9e362..4247ddd 100644
--- a/res/ExoneraTor_de.properties
+++ b/res/ExoneraTor_de.properties
@@ -22,11 +22,11 @@ summary.serverproblem.nodata.title=Server-Problem
 summary.serverproblem.nodata.body.text=Die Datenbank enthält keine Daten für das angegebene Datum. Bitte versuchen Sie es später noch einmal. Sollte dieses Problem weiterhin bestehen %s.
 summary.serverproblem.nodata.body.link=lassen Sie es uns bitte wissen
 summary.negativesamenetwork.title=Keine Ãœbereinstimmungen gefunden
-summary.negativesamenetwork.body=Es wurde kein Tor-Relay mit IP-Adresse %s am %s bzw. am Tag vorher oder nachher gefunden. Es wurden jedoch benachbarte IP-Adressen im selben /%d-Netzwerk an diesen Tagen gefunden:
+summary.negativesamenetwork.body=Es wurde kein Tor-Relay mit der IP-Adresse %s am %s bzw. am Tag vorher oder nachher gefunden. Es wurden jedoch benachbarte IP-Adressen im selben /%d-Netzwerk an diesen Tagen gefunden:
 summary.positive.title=Ãœbereinstimmungen gefunden
 summary.positive.body=Es wurde ein oder mehrere Tor-Relays mit IP-Adresse %s am %s bzw. am Tag vorher oder nachher gefunden, die den Tor-Clients bekannt waren.
 summary.negative.title=Keine Ãœbereinstimmungen gefunden
-summary.negative.body=Es wurde kein Tor-Relay mit IP-Adresse %s am %s bzw. am Tag vorher oder nachher gefunden.
+summary.negative.body=Es wurde kein Tor-Relay mit der IP-Adresse %s am %s bzw. am Tag vorher oder nachher gefunden.
 technicaldetails.heading=Technische Details
 technicaldetails.pre=Es wird nach Tor-Relays mit IP-Adresse %s am %s bzw. am Tag vorher oder nachher gesucht. Tor-Clients konnten diese Tor-Relays verwenden, um Verbindungen aufzubauen.
 technicaldetails.colheader.timestamp=Datum/Uhrzeit (UTC)
diff --git a/src/org/torproject/exonerator/ExoneraTorServlet.java b/src/org/torproject/exonerator/ExoneraTorServlet.java
index a959bc8..e7e29fe 100644
--- a/src/org/torproject/exonerator/ExoneraTorServlet.java
+++ b/src/org/torproject/exonerator/ExoneraTorServlet.java
@@ -260,9 +260,10 @@ public class ExoneraTorServlet extends HttpServlet {
 
   /* Helper methods for handling the request. */
 
-  private String parseIpParameter(String ipParameter) {
+  private String parseIpParameter(String passedIpParameter) {
     String relayIP = null;
-    if (ipParameter != null && ipParameter.length() > 0) {
+    if (passedIpParameter != null && passedIpParameter.length() > 0) {
+      String ipParameter = passedIpParameter.trim();
       Pattern ipv4AddressPattern = Pattern.compile(
           "^([01]?\\d\\d?|2[0-4]\\d|25[0-5])\\." +
           "([01]?\\d\\d?|2[0-4]\\d|25[0-5])\\." +
@@ -314,12 +315,15 @@ public class ExoneraTorServlet extends HttpServlet {
     return relayIP;
   }
 
-  private String parseTimestampParameter(String timestampParameter) {
+  private String parseTimestampParameter(
+      String passedTimestampParameter) {
     String timestampStr = "";
     SimpleDateFormat dateFormat = new SimpleDateFormat("yyyy-MM-dd");
     dateFormat.setTimeZone(TimeZone.getTimeZone("UTC"));
     dateFormat.setLenient(false);
-    if (timestampParameter != null && timestampParameter.length() > 0) {
+    if (passedTimestampParameter != null &&
+        passedTimestampParameter.length() > 0) {
+      String timestampParameter = passedTimestampParameter.trim();
       try {
         long timestamp = dateFormat.parse(timestampParameter).getTime();
         timestampStr = dateFormat.format(timestamp);
@@ -696,7 +700,8 @@ public class ExoneraTorServlet extends HttpServlet {
         + "</div><!-- row -->\n",
         rb.getString("summary.heading"),
         rb.getString("summary.invalidparams.invalidip.title"),
-        String.format("summary.invalidparams.invalidip.body",
+        String.format(
+            rb.getString("summary.invalidparams.invalidip.body"),
             escapedIpParameter, "\"a.b.c.d\"", "\"[a:b:c:d:e:f:g:h]\""));
   }
 
@@ -721,7 +726,8 @@ public class ExoneraTorServlet extends HttpServlet {
         + "</div><!-- row -->\n",
         rb.getString("summary.heading"),
         rb.getString("summary.invalidparams.invalidtimestamp.title"),
-        String.format("summary.invalidparams.invalidtimestamp.body",
+        String.format(
+            rb.getString("summary.invalidparams.invalidtimestamp.body"),
             escapedTimestampParameter, "\"YYYY-MM-DD\""));
   }