Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bugfix provided by #4225 needs wider scope #4332

Closed
davidburstromspotify opened this issue Nov 25, 2021 · 1 comment
Closed

Bugfix provided by #4225 needs wider scope #4332

davidburstromspotify opened this issue Nov 25, 2021 · 1 comment
Labels
Milestone

Comments

@davidburstromspotify
Copy link
Contributor

Expected Behavior

#4225 fixed the issue for JDK 1.8, but it turns out it's required also for JDK 11

Observed Behavior

The dreaded zip file closed is still present in JDK 11

Caused by: io.gitlab.arturbosch.detekt.api.Config$InvalidConfigurationError: Provided configuration file is invalid: Structure must be from type Map<String,Any>!
        at io.gitlab.arturbosch.detekt.core.config.YamlConfig$Companion.load(YamlConfig.kt:85)
        at io.gitlab.arturbosch.detekt.core.config.YamlConfig$Companion.loadResource(YamlConfig.kt:64)
Caused by: org.yaml.snakeyaml.error.YAMLException: java.io.IOException: Stream closed
        at org.yaml.snakeyaml.reader.StreamReader.update(StreamReader.java:218)
        at org.yaml.snakeyaml.reader.StreamReader.ensureEnoughData(StreamReader.java:176)
Caused by: java.io.IOException: Stream closed
        at java.base/java.util.zip.InflaterInputStream.ensureOpen(InflaterInputStream.java:68)
        at java.base/java.util.zip.InflaterInputStream.read(InflaterInputStream.java:143)
        at java.base/java.io.FilterInputStream.read(FilterInputStream.java:133)
        at java.base/sun.nio.cs.StreamDecoder.readBytes(StreamDecoder.java:284)

Steps to Reproduce

Run Detekt in parallel within the same VM.

Your Environment

  • Version of detekt used: 1.19.0-RC2
  • Version of Gradle used (if applicable): 7.3
  • Operating System and version: macOS Monterrey
@cortinico cortinico changed the title Bugfix provided by https://github.com/detekt/detekt/pull/4225 needs wider scope Bugfix provided by #4225 needs wider scope Nov 25, 2021
@cortinico cortinico added this to the 1.19.0 milestone Nov 25, 2021
@cortinico
Copy link
Member

Thanks for the report @davidburstromspotify
I think we can extend the fix introduced in #4225 to all the JDKs.

  • Version of detekt used: 1.19.0-RC2

Ideally we could include this inside 1.19.0.

davidburstromspotify added a commit to davidburstromspotify/detekt that referenced this issue Nov 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants