Fix leaking session
authorMathieu Baudier <mbaudier@argeo.org>
Tue, 11 Dec 2012 10:09:36 +0000 (10:09 +0000)
committerMathieu Baudier <mbaudier@argeo.org>
Tue, 11 Dec 2012 10:09:36 +0000 (10:09 +0000)
git-svn-id: https://svn.argeo.org/slc/trunk@5949 4cfe0d0a-d680-48aa-b62c-e0a02a3f76cc

plugins/org.argeo.slc.client.ui.dist/src/main/java/org/argeo/slc/client/ui/dist/commands/AddRepository.java

index f7c8a92ace2b5fe5fa0b56dc9e61bfc106be0d8e..4d46022e022e5730d005600c14a685a821e4818d 100644 (file)
@@ -173,8 +173,9 @@ public class AddRepository extends AbstractHandler implements ArgeoNames,
 
                @Override
                protected void okPressed() {
+                       Session nodeSession = null;
                        try {
-                               Session nodeSession = nodeRepository.login();
+                               nodeSession = nodeRepository.login();
                                String reposPath = UserJcrUtils.getUserHome(nodeSession)
                                                .getPath() + RepoConstants.REPOSITORIES_BASE_PATH;
 
@@ -204,6 +205,8 @@ public class AddRepository extends AbstractHandler implements ArgeoNames,
                                super.okPressed();
                        } catch (Exception e) {
                                ErrorFeedback.show("Cannot add remote repository", e);
+                       } finally {
+                               JcrUtils.logoutQuietly(nodeSession);
                        }
                }