fix: replace URL with URI in Set to avoid DNS lookups

[sonarqube-agent]

Changed Set to Set to prevent unintended DNS lookups during equals and hashCode operations. Using URI instead of URL in collections eliminates blocking network calls that can impact performance and reliability.

View Project in SonarCloud

---

Fixed Issues

java:S2112 - Use the URI class instead. • MAJOR • View issue

Location: core/src/main/java/com/webcohesion/enunciate/Enunciate.java:588

Why is this an issue?

The equals and hashCode methods of java.net.URL may trigger a name service lookup (typically DNS) to resolve the hostname or IP address. Depending on the configuration, and network status, this lookup can be time-consuming.

What changed

This is the direct fix for the issue of using java.net.URL in a Set, which can trigger DNS lookups during equals/hashCode operations. It changes the local variable declaration from Set to Set, eliminating the problematic URL-based Set at the exact line flagged by the static analysis warning.

--- a/core/src/main/java/com/webcohesion/enunciate/Enunciate.java
+++ b/core/src/main/java/com/webcohesion/enunciate/Enunciate.java
@@ -588,1 +588,1 @@ public class Enunciate implements Runnable {
-      Set<URL> sourceFiles = getSourceFileURLs();
+      Set<URI> sourceFiles = getSourceFileURLs();

Have a suggestion or found an issue? Share your feedback here.

---

SonarQube Remediation Agent uses AI. Check for mistakes.

[sonarqubecloud]

SonarQube reviewer guide

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud