Fix #4332 by widening the scope to all JDKs (#4333)

detekt · Nov 25, 2021 · a1150bc · a1150bc
1 parent 03f9b97
commit a1150bc
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/detekt-tooling/src/main/kotlin/io/github/detekt/tooling/internal/Resources.kt b/detekt-tooling/src/main/kotlin/io/github/detekt/tooling/internal/Resources.kt
@@ -7,11 +7,12 @@ fun URL.openSafeStream(): InputStream {
     return openConnection()
         /*
          * Due to https://bugs.openjdk.java.net/browse/JDK-6947916 and https://bugs.openjdk.java.net/browse/JDK-8155607,
-         * it is necessary to disallow caches to maintain stability on JDK 8. Otherwise, simultaneous invocations of
-         * Detekt in the same VM can fail spuriously. A similar bug is referenced in
+         * it is necessary to disallow caches to maintain stability on JDK 8 and 11 (and possibly more).
+         * Otherwise, simultaneous invocations of Detekt in the same VM can fail spuriously. A similar bug is referenced in
          * https://github.com/detekt/detekt/issues/3396. The performance regression is likely unnoticeable.
+         * Due to https://github.com/detekt/detekt/issues/4332 it is included for all JDKs.
          */
-        .apply { if (System.getProperty("java.specification.version") == "1.8") useCaches = false }
+        .apply { useCaches = false }
         .getInputStream()
 }