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

fixes unclosed scanner #7

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

pzzz
Copy link

@pzzz pzzz commented Jan 22, 2019

Hi all, I stumbled upon a scanner that remains unclosed which could possibly leak resources. The following PR fixes this.

Peter Schuller added 4 commits January 22, 2019 08:40
As tests were running properly on my machine I did some research.
https://discuss.circleci.com/t/the-forked-vm-terminated-without-properly-saying-goodbye-while-running-tests/26246
indicated that changes to the image might be the reason for the failing
build. To verify this I selected the last known working configuration in
this commit.
Older docker image is not available anymore.
The issue is also being discussed at the debian mailing list
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=911925
They either suggest a downgrade of java or a not to use the
system class loader. I've added configuration for the latter.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant