Skip to content

Commit

Permalink
bugfix: fixed leak by popping current context when application prep f…
Browse files Browse the repository at this point in the history
…ails, or app is not run on current instance
  • Loading branch information
lprimak committed Dec 4, 2024
1 parent d2fbb5a commit 734fc05
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -604,6 +604,7 @@ public void actOn(Logger logger) {
if (report.getActionExitCode() != ActionReport.ExitCode.SUCCESS) {
context.postDeployClean(false /* not final clean-up yet */);
events.send(new Event<>(Deployment.DEPLOYMENT_FAILURE, context));
currentDeploymentContext.get().pop();
}
}
ApplicationDeployment depl = new ApplicationDeployment(appInfo, context);
Expand Down Expand Up @@ -646,8 +647,8 @@ public void initialize(ApplicationInfo appInfo, Collection<? extends Sniffer> sn
events.send(new Event<>(Deployment.DEPLOYMENT_SUCCESS, appInfo));
}
}
currentDeploymentContext.get().pop();
}
currentDeploymentContext.get().pop();
}

@Override
Expand Down

0 comments on commit 734fc05

Please sign in to comment.