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

J21tc9 #227

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
4 changes: 3 additions & 1 deletion Model/lib/conifer/roles/conifer/vars/default.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@ _userdb_link_map:
apicommbetan: betan.login_comment
apicommdevs: devs.login_comment
apicommdevn: devn.login_comment
apicommd: devn.login_comment # local dev
apicommlegacys: legacys.login_comment
apicommlegacyn: legacyn.login_comment
cecomms: ceprods.login_comment
Expand All @@ -51,7 +52,8 @@ _userdb_link_map:
_acctdb_link_map:
acctdbn: acctdbN.profile
acctdbs: acctdbS.profile
acctdb: vm.acctdb # savm
acctdb: acctdbN.profile # local dev
# acctdb: vm.acctdb # savm

modelconfig_modelName: =c= WDK model name, e.g. ToxoDB

Expand Down
7 changes: 4 additions & 3 deletions Model/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@

<dependency>
<groupId>javax.servlet</groupId>
<artifactId>servlet-api</artifactId>
<artifactId>javax.servlet-api</artifactId>
<scope>provided</scope>
</dependency>

Expand Down Expand Up @@ -151,10 +151,11 @@
</dependency>

<!-- Contains context listener used to prevent classloader memory leaks (used by web.xmls) -->
<!-- This is throwing errors and want to eliminate it as a possible error cause
<dependency>
<groupId>se.jiderhamn.classloader-leak-prevention</groupId>
<artifactId>classloader-leak-prevention-servlet</artifactId>
</dependency>
<artifactId>classloader-leak-prevention-servlet3</artifactId>
</dependency>-->

<dependency>
<groupId>junit</groupId>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
import java.util.Properties;

import org.apache.log4j.Logger;
import org.gusdb.fgputil.db.platform.SupportedPlatform;
import org.gusdb.fgputil.db.pool.ConnectionPoolConfig;
import org.gusdb.fgputil.db.pool.DbDriverInitializer;
import org.gusdb.fgputil.db.runner.SQLRunner;
Expand Down Expand Up @@ -55,8 +56,10 @@ public void closeConnection(Connection connection, ConnectionPoolConfig dbConfig
if (dbConfig instanceof ModelConfigAppDB) {
connection.setAutoCommit(true);
ModelConfigAppDB appDbConfig = (ModelConfigAppDB)dbConfig;
closeDbLink(connection, appDbConfig.getUserDbLink());
closeDbLink(connection, appDbConfig.getAcctDbLink());
if (appDbConfig.getPlatformEnum() == SupportedPlatform.ORACLE) {
closeDbLink(connection, appDbConfig.getUserDbLink());
closeDbLink(connection, appDbConfig.getAcctDbLink());
}
}
// set auto-commit back to what it was
connection.setAutoCommit(incomingAutoCommit);
Expand Down