Repository: crawler-commons/crawler-commons
Pull request #481 (2024-11-04 20:11:50)

Bump org.apache.maven.plugins:maven-javadoc-plugin from 3.10.1 to 3.11.1
Bumps [org.apache.maven.plugins:maven-javadoc-plugin](https://github.com/apache/maven-javadoc-plugin) from 3.10.1 to 3.11.1.
<details>
<summary>Commits</summary>
<ul>
<li><a href="https://github.com/apache/maven-javadoc-plugin/commit/619650c667070ef114d571c0f075ce456ace1cf8"><code>619650c</code></a> [maven-release-plugin] prepare release maven-javadoc-plugin-3.11.1</li>
<li><a href="https://github.com/apache/maven-javadoc-plugin/commit/e314da0226d100eae39d96d7a0cbdad799879dca"><code>e314da0</code></a> [MJAVADOC-821] Align toolchain discovery code with Maven Compiler Plugin</li>
<li><a href="https://github.com/apache/maven-javadoc-plugin/commit/62a686180915de1e3c0a2f71081776eacfd1d4c3"><code>62a6861</code></a> [MJAVADOC-820] [REGRESSION] MJAVADOC-787 was merged incompletely</li>
<li><a href="https://github.com/apache/maven-javadoc-plugin/commit/d1090c50ce66e10ab0adcadb996b0fe790c900b6"><code>d1090c5</code></a> [maven-release-plugin] prepare for next development iteration</li>
<li><a href="https://github.com/apache/maven-javadoc-plugin/commit/ee030f719f4bc7df1671f6dfe5993f778b57a504"><code>ee030f7</code></a> [maven-release-plugin] prepare release maven-javadoc-plugin-3.11.0</li>
<li><a href="https://github.com/apache/maven-javadoc-plugin/commit/6c5fdc06d76fdd429c4d3ae01e70be8917c64bea"><code>6c5fdc0</code></a> [MJAVADOC-819] Align archive generation code with Maven Source Plugin</li>
<li><a href="https://github.com/apache/maven-javadoc-plugin/commit/3a90de597adff7c8f8c4a420400f24c7386e9880"><code>3a90de5</code></a> [MJAVADOC-787] Automatic detection of release option for JDK &lt; 9</li>
<li><a href="https://github.com/apache/maven-javadoc-plugin/commit/373172d8782ad011cc14e20ac78a84d032ad1d4b"><code>373172d</code></a> [MJAVADOC-817] Upgrade to Doxia 2.0.0 GA Stack</li>
<li><a href="https://github.com/apache/maven-javadoc-plugin/commit/ba266c0fe11a822bf3fb19c365a174246eae29f9"><code>ba266c0</code></a> Fix SCM tag</li>
<li><a href="https://github.com/apache/maven-javadoc-plugin/commit/5775ce13455d565bd10b15cbeb94310074dee8a3"><code>5775ce1</code></a> Fix typo</li>
<li>Additional commits viewable in <a href="https://github.com/apache/maven-javadoc-plugin/compare/maven-javadoc-plugin-3.10.1...maven-javadoc-plugin-3.11.1">compare view</a></li>
</ul>
</details>
<br />


[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=org.apache.maven.plugins:maven-javadoc-plugin&package-manager=maven&previous-version=3.10.1&new-version=3.11.1)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
- `@dependabot show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)


</details>


Patch:
diff --git a/pom.xml b/pom.xml
index 41bb98a..0ee4d27 100644
--- a/pom.xml
+++ b/pom.xml
@@ -272,13 +272,7 @@
 				<artifactId>maven-javadoc-plugin</artifactId>
 				<version>${maven-javadoc-plugin.version}</version>
 				<configuration>
-					<links>
-						<link>https://docs.oracle.com/javase/8/docs/api/</link>
-					</links>
 					<source>${javac.src.version}</source>
-					<additionalJOptions>
-						<additionalJOption>${additionalJavadocOpts}</additionalJOption>
-					</additionalJOptions>
 				</configuration>
 				<executions>
 					<execution>
@@ -339,7 +333,6 @@
 								</goals>
 								<configuration>
 									<quiet>true</quiet>
-									<additionalJOption>${additionalJavadocOpts}</additionalJOption>
 									<archive>
 										<manifest>
 											<addDefaultImplementationEntries>true</addDefaultImplementationEntries>
@@ -378,15 +371,6 @@
 				</plugins>
 			</build>
 		</profile>
-		<profile>
-			<id>javadoc-no-module-directories</id>
-			<activation>
-				<jdk>[11,12)</jdk>
-			</activation>
-			<properties>
-				<additionalJavadocOpts>--no-module-directories</additionalJavadocOpts>
-			</properties>
-		</profile>
 	</profiles>
 
 	<properties>
@@ -404,13 +388,13 @@
 		<maven-surfire-plugin.version>3.5.1</maven-surfire-plugin.version>
 		<maven-release-plugin.version>3.1.1</maven-release-plugin.version>
 		<maven-source-plugin.version>3.3.1</maven-source-plugin.version>
-		<maven-javadoc-plugin.version>3.10.1</maven-javadoc-plugin.version>
+		<maven-javadoc-plugin.version>3.11.1</maven-javadoc-plugin.version>
 		<maven-gpg-plugin.version>3.2.7</maven-gpg-plugin.version>
 		<apache-rat-plugin.version>0.8</apache-rat-plugin.version>
 		<maven-assembly-plugin.version>2.2.2</maven-assembly-plugin.version>
 		<maven-deploy-plugin.version>3.1.3</maven-deploy-plugin.version>
 		<checksum-maven-plugin.version>1.4</checksum-maven-plugin.version>
-		<maven.download.plugin.version>1.11.1</maven.download.plugin.version>
+		<maven.download.plugin.version>1.11.2</maven.download.plugin.version>
 
 		<!-- General Properties -->
 		<implementation.build>${scmBranch}@r${buildNumber}</implementation.build>
diff --git a/src/main/java/crawlercommons/domains/EffectiveTldFinder.java b/src/main/java/crawlercommons/domains/EffectiveTldFinder.java
index df0ec91..ee90d27 100644
--- a/src/main/java/crawlercommons/domains/EffectiveTldFinder.java
+++ b/src/main/java/crawlercommons/domains/EffectiveTldFinder.java
@@ -1,12 +1,12 @@
 /**
  * Copyright 2016 Crawler-Commons
- * 
+ *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *     http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -43,7 +43,7 @@ import org.slf4j.LoggerFactory;
  * class implements the
  * <a href="https://publicsuffix.org/list/">publicsuffix.org ruleset</a> and
  * uses a copy of the public suffix list.
- * 
+ *
  * For more information, see
  * <ul>
  * <li><a href="https://www.publicsuffix.org/">publicsuffix.org</a></li>
@@ -54,7 +54,7 @@ import org.slf4j.LoggerFactory;
  * Service</a>: for historic reasons the class name stems from the term
  * &quot;effective top-level domain&quot; (eTLD)</li>
  * </ul>
- * 
+ *
  * EffectiveTldFinder loads the public suffix list as file
  * "effective_tld_names.dat" from the Java classpath. Make sure your classpath
  * does not contain any other file with the same name, eg. an outdated list
@@ -75,9 +75,9 @@ import org.slf4j.LoggerFactory;
  * >https://raw.githubusercontent.com/publicsuffix/list/master/
  * public_suffix_list.dat</a></li>
  * </ul>
- * 
+ *
  * <h2>ICANN vs. Private Domains</h2>
- * 
+ *
  * The <a href="https://publicsuffix.org/list/">public suffix list (see section
  * &quot;divisions&quot;)</a> is subdivided into &quot;ICANN&quot; and
  * &quot;PRIVATE&quot; domains. To restrict the EffectiveTldFinder to
@@ -87,7 +87,7 @@ import org.slf4j.LoggerFactory;
  * {@link EffectiveTldFinder#getEffectiveTLD(String, boolean)}. This will
  * exclude the eTLDs from the PRIVATE domain section of the public suffix list
  * while a domain or eTLD is matched.
- * 
+ *
  */
 public class EffectiveTldFinder {
     private static final Logger LOGGER = LoggerFactory.getLogger(EffectiveTldFinder.class);
@@ -104,7 +104,7 @@ public class EffectiveTldFinder {
      * (applies to domain names as well), cf.
      * https://tools.ietf.org/html/rfc1034#section-3.1 and
      * https://en.wikipedia.org/wiki/Hostname#Restrictions_on_valid_hostnames
-     * 
+     *
      * Note: We only have to validate domain names and not the host names passed
      * as input. For domain names a verification of the segment length also
      * implies that the entire domain names stays in the limit of 253
@@ -134,7 +134,7 @@ public class EffectiveTldFinder {
 
     /**
      * Get singleton instance of EffectiveTldFinder with default configuration.
-     * 
+     *
      * @return singleton instance of EffectiveTldFinder
      */
     public static EffectiveTldFinder getInstance() {
@@ -146,7 +146,7 @@ public class EffectiveTldFinder {
 
     /**
      * (Re)initialize EffectiveTldFinder with custom public suffix list.
-     * 
+     *
      * @param effectiveTldDataStream
      *            content of public suffix list as input stream
      * @return true if (re)initialization was successful
@@ -194,7 +194,7 @@ public class EffectiveTldFinder {
     /**
      * Get EffectiveTLD for host name using the singleton instance of
      * EffectiveTldFinder.
-     * 
+     *
      * @param hostname
      *            the hostname for which to find the {@link EffectiveTLD}
      * @return the {@link EffectiveTLD}
@@ -206,7 +206,7 @@ public class EffectiveTldFinder {
     /**
      * Get EffectiveTLD for host name using the singleton instance of
      * EffectiveTldFinder.
-     * 
+     *
      * @param hostname
      *            the hostname for which to find the {@link EffectiveTLD}
      * @param excludePrivate
@@ -225,7 +225,7 @@ public class EffectiveTldFinder {
     /**
      * Find EffectiveTLD and offset in host name using the singleton instance of
      * EffectiveTldFinder.
-     * 
+     *
      * @param hostname
      *            the hostname for which to find the {@link EffectiveTLD}
      * @param excludePrivate
@@ -243,11 +243,15 @@ public class EffectiveTldFinder {
                 if (excludePrivate && foundTld.isPrivate) {
                     continue;
                 }
-                if (offset == 0 || foundTld.isException() || !foundTld.isWild()) {
+                if (offset == 0 || foundTld.isException() || !foundTld.isWildcard()) {
+                    if (foundTld.idn != null) {
+                        foundTld = new EffectiveTLD(hostname.substring(offset), foundTld);
+                        return new SuffixTrie.LookupResult<EffectiveTLD>(offset, foundTld);
+                    }
                     return res;
                 }
 
-                // wildcards create an open ETLD namespace
+                // wildcard suffixes create an open ETLD namespace
                 int wildcardOffset = hostname.lastIndexOf(DOT, offset - 2);
                 String retryTld;
                 if (wildcardOffset == -1) {
@@ -259,7 +263,7 @@ public class EffectiveTldFinder {
                 }
 
                 try {
-                    foundTld = new EffectiveTLD(retryTld, foundTld.isPrivate);
+                    foundTld = new EffectiveTLD(retryTld, foundTld);
                 } catch (IllegalArgumentException e) {
                     // retryTld contains forbidden characters
                     return null;
@@ -274,7 +278,7 @@ public class EffectiveTldFinder {
     /**
      * This method uses the effective TLD to determine which component of a FQDN
      * is the NIC-assigned domain name (aka "Paid Level Domain").
-     * 
+     *
      * @param hostname
      *            a string for which to obtain a NIC-assigned domain name
      * @return the NIC-assigned domain name or as fall-back the hostname if no
@@ -287,7 +291,7 @@ public class EffectiveTldFinder {
     /**
      * This method uses the effective TLD to determine which component of a FQDN
      * is the NIC-assigned domain name (aka "Paid Level Domain").
-     * 
+     *
      * @param hostname
      *            a string for which to obtain a NIC-assigned domain name
      * @param strict
@@ -303,7 +307,7 @@ public class EffectiveTldFinder {
     /**
      * This method uses the effective TLD to determine which component of a FQDN
      * is the NIC-assigned domain name.
-     * 
+     *
      * @param hostname
      *            a string for which to obtain a NIC-assigned domain name
      * @param strict
@@ -382,7 +386,7 @@ public class EffectiveTldFinder {
         return sb.toString();
     }
 
-    public static void help() {
+    private static void help() {
         LOGGER.error("EffectiveTldFinder [-etld] [-strict] [-excludePrivate]");
         LOGGER.error("  get domains or public suffixes for host names");
         LOGGER.error("Options:");
@@ -448,22 +452,38 @@ public class EffectiveTldFinder {
      * <li>for IDN suffixes: both the ASCII and IDN variant
      * (<code>xn--p1ai</code> and <code>рф</code>)</li>
      * <li>and the properties required to parse host/domain names given in the
-     * public suffix list (wildcard suffix, exception, in private domain
-     * section)</li>
+     * public suffix list:
+     * <ul>
+     * <li>whether it's a wildcard suffix (<code>*.kawasaki.jp</code>),</li>
+     * <li>or an exception to a wildcard rule
+     * (<code>!city.kawasaki.jp</code>,</li>
+     * <li>or whether the suffix is in the private domain section</li>
      * </ul>
+     * </li>
+     * <li>for Internationalized Domain Names (IDNs) both the Unicode and ASCII
+     * representation</li>
+     * </ul>
+     *
+     * In addition, EffectiveTLD is used to hold the result of a public suffix
+     * match returned by
+     * {@link EffectiveTldFinder#getEffectiveTLD(String, boolean)}. A match
+     * result also includes the literally matched suffix which is distinct from
+     * the public suffix in case of wildcard suffixes or, eventually, for
+     * Internationalized Domain Names (IDNs).
      */
     public static class EffectiveTLD {
 
         private boolean exception = false;
-        private boolean wild = false;
+        private boolean wildcard = false;
         private boolean isPrivate = false;
         private String domain = null;
         private String idn = null;
+        private String suffix = null; // same as domain except for matched wildcard suffixes
 
         /**
          * Parse one non-empty, non-comment line in the public suffix list and
          * hold the public suffix and its properties in the created object.
-         * 
+         *
          * @param line
          *            non-empty, non-comment line in the public suffix list
          * @param isPrivateDomain
@@ -478,7 +498,7 @@ public class EffectiveTldFinder {
                 exception = true;
                 domain = line.substring(EXCEPTION.length(), line.length());
             } else if (line.startsWith(WILD_CARD)) {
-                wild = true;
+                wildcard = true;
                 domain = line.substring(WILD_CARD.length(), line.length());
             } else {
                 domain = line;
@@ -489,15 +509,33 @@ public class EffectiveTldFinder {
                 idn = domain;
                 domain = norm;
             }
+            suffix = domain;
             isPrivate = isPrivateDomain;
         }
 
         /**
-         * Normalize a domain name: convert characters into to lowercase and
-         * encode dot-separated segments containing non-ASCII characters. Cf.
+         * Constructor for a matched public suffix.
+         *
+         * @param representation
+         *            the matched public suffix part in the host or domain name
+         * @param from
+         *            the {@link EffectiveTLD} representing the public suffix
+         */
+        private EffectiveTLD(String representation, EffectiveTLD from) {
+            domain = representation;
+            suffix = from.domain;
+            wildcard = from.wildcard;
+            exception = from.exception;
+            isPrivate = from.isPrivate;
+            idn = from.idn;
+        }
+
+        /**
+         * Normalize a domain name: convert characters to lowercase and encode
+         * dot-separated segments containing non-ASCII characters. Cf.
          * {@link #asciiConvert(String)} and {@link IDN#toASCII(String)}
-         * 
-         * @param str
+         *
+         * @param name
          *            domain name segment
          * @return normalized domain name containing only ASCII characters
          * @throws IllegalArgumentException
@@ -516,7 +554,7 @@ public class EffectiveTldFinder {
          * Generate name variants caused by Internationalized Domain Names:
          * every IDN part of a eTLD can be replaced by its punycoded ASCII
          * variant. For two-part IDN eTLDs this will generate 4 variants.
-         * 
+         *
          * @return set of variant names
          */
         public Set<String> getNameVariants() {
@@ -558,7 +596,7 @@ public class EffectiveTldFinder {
         /**
          * Converts a single domain name segment (separated by dots) to ASCII if
          * it contains non-ASCII character, cf. {@link IDN#toASCII(String)}.
-         * 
+         *
          * @param str
          *            domain name segment
          * @return ASCII "Punycode" representation of the domain name segment
@@ -582,14 +620,63 @@ public class EffectiveTldFinder {
             return true;
         }
 
+        /**
+         * @return the public suffix (effective TLD) or for match results
+         *         returned by
+         *         {@link EffectiveTldFinder#getEffectiveTLD(String, boolean)}
+         *         the literally matched suffix which can be different from the
+         *         suffix string in the public suffix list in case of wildcard
+         *         or Unicode (IDN) suffixes.
+         * @see #getSuffix()
+         * @see #getUnicodeDomain()
+         */
         public String getDomain() {
             return domain;
         }
 
+        /**
+         * @return the public suffix string from the public suffix list, for
+         *         Unicode suffixes the ASCII (Punycode) representation
+         */
+        public String getSuffix() {
+            if (wildcard) {
+                return "*." + suffix;
+            } else if (exception) {
+                return "!" + suffix;
+            }
+            return suffix;
+        }
+
+        /**
+         * @return the Unicode version of a public suffix or <code>null</code>
+         *         if it's not an IDN suffix
+         */
+        public String getUnicodeDomain() {
+            return idn;
+        }
+
+        /**
+         * @return true if the public suffix (effective TLD) is a wildcard
+         *         suffix (e.g., <code>*.kawasaki.jp</code>)
+         * @deprecated since 1.6 - replaced by {@link #isWildcard()}
+         */
+        @Deprecated
         public boolean isWild() {
-            return wild;
+            return wildcard;
         }
 
+        /**
+         * @return true if the public suffix (effective TLD) is a wildcard
+         *         suffix (e.g., <code>*.kawasaki.jp</code>)
+         */
+        public boolean isWildcard() {
+            return wildcard;
+        }
+
+        /**
+         * @return true if the public suffix (effective TLD) is an exception to
+         *         a wildcard suffix (e.g., <code>!city.kawasaki.jp</code>)
+         */
         public boolean isException() {
             return exception;
         }
@@ -598,7 +685,9 @@ public class EffectiveTldFinder {
         public String toString() {
             StringBuffer sb = new StringBuffer("[");
             sb.append("domain=").append(domain).append(",");
-            sb.append("wild=").append(wild).append(",");
+            sb.append("suffix=").append(getSuffix()).append(",");
+            sb.append("idn=").append(idn).append(",");
+            sb.append("wildcard=").append(wildcard).append(",");
             sb.append("exception=").append(exception).append(",");
             sb.append("private=").append(isPrivate).append("]");
             return sb.toString();


Test patch:
diff --git a/src/test/java/crawlercommons/domains/EffectiveTldFinderTest.java b/src/test/java/crawlercommons/domains/EffectiveTldFinderTest.java
index 65b3a8f..bf5957e 100644
--- a/src/test/java/crawlercommons/domains/EffectiveTldFinderTest.java
+++ b/src/test/java/crawlercommons/domains/EffectiveTldFinderTest.java
@@ -1,12 +1,12 @@
 /**
  * Copyright 2016 Crawler-Commons
- * 
+ *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *     http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -52,7 +52,7 @@ public class EffectiveTldFinderTest {
         EffectiveTLD etld = null;
         etld = EffectiveTldFinder.getEffectiveTLD("blogs.yahoo.co.jp");
         assertFalse(etld.isException());
-        assertFalse(etld.isWild());
+        assertFalse(etld.isWildcard());
         assertEquals("co.jp", etld.getDomain());
     }
 
@@ -61,23 +61,29 @@ public class EffectiveTldFinderTest {
         EffectiveTLD etld = null;
         etld = EffectiveTldFinder.getEffectiveTLD("uk");
         assertFalse(etld.isException());
-        assertFalse(etld.isWild());
+        assertFalse(etld.isWildcard());
         etld = EffectiveTldFinder.getEffectiveTLD("bbc.co.uk");
         assertFalse(etld.isException());
-        assertFalse(etld.isWild());
+        assertFalse(etld.isWildcard());
         assertEquals("co.uk", etld.getDomain());
         etld = EffectiveTldFinder.getEffectiveTLD("www.bbc.co.uk");
         assertFalse(etld.isException());
-        assertFalse(etld.isWild());
+        assertFalse(etld.isWildcard());
         assertEquals("co.uk", etld.getDomain());
         etld = EffectiveTldFinder.getEffectiveTLD("anything.uk");
         assertEquals("uk", etld.getDomain());
         etld = EffectiveTldFinder.getEffectiveTLD("northleamingtonschool.warwickshire.sch.uk");
         assertEquals("warwickshire.sch.uk", etld.getDomain());
-        assertFalse(etld.isWild());
+        assertTrue(etld.isWildcard());
+        assertEquals("*.sch.uk", etld.getSuffix());
         etld = EffectiveTldFinder.getEffectiveTLD("sch.uk");
         assertEquals("sch.uk", etld.getDomain());
-        assertTrue(etld.isWild());
+        assertTrue(etld.isWildcard());
+        assertEquals("*.sch.uk", etld.getSuffix());
+        etld = EffectiveTldFinder.getEffectiveTLD("abc.def.bd");
+        assertEquals("def.bd", etld.getDomain());
+        assertTrue(etld.isWildcard());
+        assertEquals("*.bd", etld.getSuffix());
     }
 
     @Test
@@ -85,23 +91,24 @@ public class EffectiveTldFinderTest {
         EffectiveTLD etld = null;
         etld = EffectiveTldFinder.getEffectiveTLD("parliament.uk");
         assertFalse(etld.isException());
-        assertFalse(etld.isWild());
+        assertFalse(etld.isWildcard());
         // should be parliament.uk
         assertEquals("uk", etld.getDomain());
         etld = EffectiveTldFinder.getEffectiveTLD("www.parliament.uk");
         assertFalse(etld.isException());
-        assertFalse(etld.isWild());
+        assertFalse(etld.isWildcard());
         // should be parliament.uk
         assertEquals("uk", etld.getDomain());
         etld = EffectiveTldFinder.getEffectiveTLD("hokkaido.jp");
         assertFalse(etld.isException());
-        assertFalse(etld.isWild());
+        assertFalse(etld.isWildcard());
         assertEquals("hokkaido.jp", etld.getDomain());
         // test a Japanese city
         etld = EffectiveTldFinder.getEffectiveTLD("www.city.kawasaki.jp");
         assertTrue(etld.isException());
-        assertFalse(etld.isWild());
+        assertFalse(etld.isWildcard());
         assertEquals("city.kawasaki.jp", etld.getDomain());
+        assertEquals("!city.kawasaki.jp", etld.getSuffix());
         assertEquals("city.kawasaki.jp", EffectiveTldFinder.getAssignedDomain("www.city.kawasaki.jp"));
         assertEquals(".kawasaki.jp", EffectiveTldFinder.getAssignedDomain(".kawasaki.jp"));
         assertNull(EffectiveTldFinder.getAssignedDomain(".kawasaki.jp", true));
@@ -183,13 +190,38 @@ public class EffectiveTldFinderTest {
         assertEquals("бесплатныеобъявления.рф", ad);
         ad = EffectiveTldFinder.getAssignedDomain("xn--90a1af.xn--80abbembcyvesfij3at4loa4ff.xn--p1ai", true);
         assertEquals("xn--80abbembcyvesfij3at4loa4ff.xn--p1ai", ad);
-        // rare but possible mixed use of UTF-8 and Punycode
+        // rare but possible mixed use of Unicode and ASCII/Punycode
         ad = EffectiveTldFinder.getAssignedDomain("xn--90a1af.бесплатныеобъявления.рф", true);
         assertEquals("бесплатныеобъявления.рф", ad);
         ad = EffectiveTldFinder.getAssignedDomain("xn--90a1af.xn--80abbembcyvesfij3at4loa4ff.рф", true);
         assertEquals("xn--80abbembcyvesfij3at4loa4ff.рф", ad);
     }
 
+    @Test
+    public final void testIDNEtld() throws Exception {
+        EffectiveTLD etld = null;
+        etld = EffectiveTldFinder.getEffectiveTLD("спб.бесплатныеобъявления.рф", true);
+        assertEquals("рф", etld.getDomain());
+        assertEquals("рф", etld.getUnicodeDomain());
+        etld = EffectiveTldFinder.getEffectiveTLD("xn--90a1af.xn--80abbembcyvesfij3at4loa4ff.xn--p1ai", true);
+        assertEquals("xn--p1ai", etld.getDomain());
+        assertEquals("рф", etld.getUnicodeDomain());
+        // rare but possible mixed use of Unicode and ASCII/Punycode
+        etld = EffectiveTldFinder.getEffectiveTLD("xn--90a1af.бесплатныеобъявления.рф", true);
+        assertEquals("рф", etld.getDomain());
+        assertEquals("рф", etld.getUnicodeDomain());
+        etld = EffectiveTldFinder.getEffectiveTLD("xn--90a1af.xn--80abbembcyvesfij3at4loa4ff.рф", true);
+        assertEquals("рф", etld.getDomain());
+        assertEquals("рф", etld.getUnicodeDomain());
+
+        etld = EffectiveTldFinder.getEffectiveTLD("中国政府网.政务", true);
+        assertEquals("政务", etld.getDomain());
+        assertEquals("政务", etld.getUnicodeDomain());
+        etld = EffectiveTldFinder.getEffectiveTLD("xn--fiqs8sirgfmhq98a.xn--zfr164b", true);
+        assertEquals("xn--zfr164b", etld.getDomain());
+        assertEquals("政务", etld.getUnicodeDomain());
+    }
+
     @Test
     public final void testStrictDomain() throws Exception {
         String ad = null;