commit 225d504fcf7b36df7fb9f0f36c5dc07c1d4cb6ed Author: Karsten Loesing karsten.loesing@gmx.net Date: Mon Aug 20 20:47:55 2018 +0200
Remove redundant escape in regular expressions.
Turns out that characters inside square brackets don't need to be escaped. In fact, "^[0-9\.]{7,15}$" does not only match valid IPv4 addresses but also strings like "1\2\3\4". --- .../java/org/torproject/onionoo/server/ResourceServlet.java | 10 +++++----- .../java/org/torproject/onionoo/updater/LookupService.java | 2 +- 2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/src/main/java/org/torproject/onionoo/server/ResourceServlet.java b/src/main/java/org/torproject/onionoo/server/ResourceServlet.java index 8f956a6..7567233 100644 --- a/src/main/java/org/torproject/onionoo/server/ResourceServlet.java +++ b/src/main/java/org/torproject/onionoo/server/ResourceServlet.java @@ -76,7 +76,7 @@ public class ResourceServlet extends HttpServlet { + "offset,fields").split(",")));
private static String ipv6AddressPatternString = - "^\[?[0-9a-fA-F:\.]{1,39}\]?$"; + "^\[?[0-9a-fA-F:.]{1,39}\]?$";
private static Pattern ipv6AddressPattern = Pattern.compile(ipv6AddressPatternString); @@ -424,14 +424,14 @@ public class ResourceServlet extends HttpServlet { }
private static Pattern searchQueryStringPattern = - Pattern.compile("(?:.*[\?&])*?" // lazily skip other parameters + Pattern.compile("(?:.*[?&])*?" // lazily skip other parameters + "search=([\p{Graph} &&[^&]]+)" // capture parameter + "(?:&.*)*"); // skip remaining parameters
private static Pattern searchParameterPattern = Pattern.compile("^\$?[0-9a-fA-F]{1,40}$|" /* Hex fingerprint. */ + "^[0-9a-zA-Z+/]{1,27}$|" /* Base64 fingerprint. */ - + "^[0-9a-zA-Z\.]{1,19}$|" /* Nickname or IPv4 address. */ + + "^[0-9a-zA-Z.]{1,19}$|" /* Nickname or IPv4 address. */ + ipv6AddressPatternString + "|" /* IPv6 address. */ + "^[a-zA-Z_]+:"?[\p{Graph} ]+"?$"); /* Qualified search term. */
@@ -630,7 +630,7 @@ public class ResourceServlet extends HttpServlet { }
private static Pattern versionParameterPattern = - Pattern.compile("^[0-9a-zA-Z\.-]+$"); + Pattern.compile("^[0-9a-zA-Z.-]+$");
private String parseVersionParameter(String parameter) { if (!versionParameterPattern.matcher(parameter).matches()) { @@ -651,7 +651,7 @@ public class ResourceServlet extends HttpServlet { }
private static Pattern hostNameParameterPattern = - Pattern.compile("^[0-9A-Za-z_\.\-]+$"); + Pattern.compile("^[0-9A-Za-z_.\-]+$");
private String parseHostNameParameter(String parameter) { if (!hostNameParameterPattern.matcher(parameter).matches()) { diff --git a/src/main/java/org/torproject/onionoo/updater/LookupService.java b/src/main/java/org/torproject/onionoo/updater/LookupService.java index 9c07e68..976a9ca 100644 --- a/src/main/java/org/torproject/onionoo/updater/LookupService.java +++ b/src/main/java/org/torproject/onionoo/updater/LookupService.java @@ -70,7 +70,7 @@ public class LookupService { this.hasAllFiles = true; }
- private Pattern ipv4Pattern = Pattern.compile("^[0-9\.]{7,15}$"); + private Pattern ipv4Pattern = Pattern.compile("^[0-9.]{7,15}$");
private long parseAddressString(String addressString) { long addressNumber = -1L;