Skip to content

Fix resource leak in ZipUtils#10815

Merged
tsmaeder merged 3 commits intoeclipse-che:5730_java_ls_pocfrom
tsmaeder:10764_fix_project_zipping
Aug 17, 2018
Merged

Fix resource leak in ZipUtils#10815
tsmaeder merged 3 commits intoeclipse-che:5730_java_ls_pocfrom
tsmaeder:10764_fix_project_zipping

Conversation

@tsmaeder
Copy link
Copy Markdown
Contributor

What does this PR do?

Fixes a resource leak when adding directories to a zip file with ZipUtils.

What issues does this PR fix or reference?

10764

Signed-off-by: Thomas Mäder <tmader@redhat.com>
Signed-off-by: Thomas Mäder <tmader@redhat.com>
Signed-off-by: Thomas Mäder <tmader@redhat.com>
@tsmaeder tsmaeder added kind/bug Outline of a bug - must adhere to the bug report template. status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. target/branch Indicates that a PR will be merged into a branch other than master. e2e-test/failure Issues that is related to a test failures reported by our CI platform and our QE. labels Aug 17, 2018
@tsmaeder tsmaeder requested a review from skabashnyuk as a code owner August 17, 2018 08:48
@tsmaeder tsmaeder merged commit 7bebd1c into eclipse-che:5730_java_ls_poc Aug 17, 2018
@benoitf benoitf removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Aug 17, 2018
tsmaeder added a commit that referenced this pull request Aug 22, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Aug 29, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tolusha pushed a commit that referenced this pull request Sep 5, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Sep 13, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Sep 20, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Sep 26, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Oct 1, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Oct 5, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Oct 12, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Oct 16, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Oct 17, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Oct 17, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Oct 17, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Oct 17, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Oct 17, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

e2e-test/failure Issues that is related to a test failures reported by our CI platform and our QE. kind/bug Outline of a bug - must adhere to the bug report template. target/branch Indicates that a PR will be merged into a branch other than master.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants