Skip to content

Fixing org.glassfish.admin.payload.PayloadFilesManager memory leak #7909

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.

Sign up for GitHub

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

Jump to bottom
Open
svendiedrichsen wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Fixing org.glassfish.admin.payload.PayloadFilesManager memory leak #7909

svendiedrichsen wants to merge 1 commit into from
+19 −19

Conversation

@svendiedrichsen
Copy link
Contributor

@svendiedrichsen svendiedrichsen commented Jan 26, 2026 •
edited
Loading

Description

using internal static class to wrap targetDir state of Temp to allow file cleanup Temp instance is GC'd

Briefly mentioned in and aiming to fix for #7849

Important Info

Blockers

none

Testing

Testing Performed

PR pipeline

Notes for Reviewers

see description

@svendiedrichsen svendiedrichsen changed the title Fixing PayloadFilesManager memory leak Fixing org.glassfish.admin.payload.PayloadFilesManager memory leak Jan 26, 2026
@svendiedrichsen
Copy link
Contributor Author

svendiedrichsen commented Jan 27, 2026 •
edited
Loading

The build error is unrelated to the change.

[ERROR] Failed to execute goal on project jaspic-common: 
Could not resolve dependencies for project org.javaee7:jaspic-common:jar:1.0-SNAPSHOT: 
Could not find artifact jakarta.authorization:jakarta.authorization-api:jar:3.0.0-M4 in central (https://repo.maven.apache.org/maven2) -> [Help 1]

@kalinchan kalinchan added the PR: CLA CLA submitted on PR by the contributor label Jan 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@lprimak lprimak lprimak approved these changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

PR: CLA CLA submitted on PR by the contributor

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@svendiedrichsen @lprimak @kalinchan