(*) Fix bug in EdgeDomain where it would permit domains with a trailing period, DNS style.

2023-12-29 16:36:01 +01:00 · 2023-12-29 16:36:01 +01:00 · 6aee27a3f1
commit 6aee27a3f1
parent ea73be6831
1 changed files with 6 additions and 0 deletions
--- a/code/common/model/src/main/java/nu/marginalia/model/EdgeDomain.java
+++ b/code/common/model/src/main/java/nu/marginalia/model/EdgeDomain.java
@ -23,6 +23,12 @@ public class EdgeDomain implements Serializable {

        host = host.toLowerCase();

+        // Remove trailing dots, which are allowed in DNS but not in URLs
+        // (though sometimes still show up in the wild)
+        while (!host.isBlank() && host.endsWith(".")) {
+            host = host.substring(0, host.length() - 1);
+        }
+
        var dot = host.lastIndexOf('.');

        if (dot < 0 || looksLikeAnIp(host)) { // IPV6 >.>