From 4ca4bd22276903d759732b0ecfb518113fdd9234 Mon Sep 17 00:00:00 2001
From: Johngreen <ramses203@naver.com>
Date: Mon, 3 Nov 2025 11:34:48 +0900
Subject: [PATCH] =?UTF-8?q?=EC=BB=A4=EB=84=A5=EC=85=98=EC=84=A4=EC=A0=95?=
 =?UTF-8?q?=20=EB=B3=B4=EC=99=84=EC=9E=91=EC=97=85?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 src/com/pms/service/CommonService.java | 45 +++++++++++++++-----------
 tomcat-conf/context.xml                | 16 ++++++++-
 2 files changed, 42 insertions(+), 19 deletions(-)

diff --git a/src/com/pms/service/CommonService.java b/src/com/pms/service/CommonService.java
index a659dff..c686f7d 100644
--- a/src/com/pms/service/CommonService.java
+++ b/src/com/pms/service/CommonService.java
@@ -341,24 +341,33 @@ public class CommonService extends BaseService {
 	 * @param fileMap
 	 */
 	public void setFileDownloadLog(HttpServletRequest request, Map fileMap){
-		HttpSession session = request.getSession();
-		PersonBean person = (PersonBean)session.getAttribute(Constants.PERSON_BEAN);
-		
-		String objId = CommonUtils.createObjId();
-		String systemName = Constants.SYSTEM_NAME;
-		String userId = CommonUtils.checkNull(person.getUserId());
-		String fileObjId = CommonUtils.checkNull(fileMap.get("OBJID"));
-		String remoteAddr = CommonUtils.checkNull(request.getRemoteAddr());
-		
-		Map paramMap = new HashMap();
-		paramMap.put("objId", objId);
-		paramMap.put("systemName", systemName);
-		paramMap.put("userId", userId);
-		paramMap.put("fileObjId", fileObjId);
-		paramMap.put("remoteAddr", remoteAddr);
-		
-		SqlSession sqlSession = SqlMapConfig.getInstance().getSqlSession();
-		sqlSession.insert("common.insertFileDownloadLog", paramMap);
+		SqlSession sqlSession = null;
+		try{
+			HttpSession session = request.getSession();
+			PersonBean person = (PersonBean)session.getAttribute(Constants.PERSON_BEAN);
+			
+			String objId = CommonUtils.createObjId();
+			String systemName = Constants.SYSTEM_NAME;
+			String userId = CommonUtils.checkNull(person.getUserId());
+			String fileObjId = CommonUtils.checkNull(fileMap.get("OBJID"));
+			String remoteAddr = CommonUtils.checkNull(request.getRemoteAddr());
+			
+			Map paramMap = new HashMap();
+			paramMap.put("objId", objId);
+			paramMap.put("systemName", systemName);
+			paramMap.put("userId", userId);
+			paramMap.put("fileObjId", fileObjId);
+			paramMap.put("remoteAddr", remoteAddr);
+			
+			sqlSession = SqlMapConfig.getInstance().getSqlSession();
+			sqlSession.insert("common.insertFileDownloadLog", paramMap);
+		}catch(Exception e){
+			e.printStackTrace();
+		}finally{
+			if(sqlSession != null){
+				sqlSession.close();
+			}
+		}
 	}
 	
 	/**
diff --git a/tomcat-conf/context.xml b/tomcat-conf/context.xml
index bb5ebcd..2aedf8c 100644
--- a/tomcat-conf/context.xml
+++ b/tomcat-conf/context.xml
@@ -5,10 +5,24 @@
 
     <!-- PLM DataSource JNDI Resource -->
     <!-- Using environment variables for database connection -->
+    <!-- 
+        개선된 Connection Pool 설정 (커넥션 누수 방지):
+        - maxTotal: 최대 커넥션 수 (기본 200 유지)
+        - maxIdle: 유휴 커넥션 수 (기본 50 유지)
+        - maxWaitMillis: 커넥션 획득 대기 시간 (무한 대기 -> 10초로 제한)
+        - testOnBorrow: 커넥션 대여 시 유효성 검사
+        - validationQuery: 유효성 검사 쿼리
+        - removeAbandonedOnBorrow: 누수된 커넥션 자동 회수 활성화
+        - removeAbandonedTimeout: 60초 이상 사용 중인 커넥션 회수
+        - logAbandoned: 누수된 커넥션 로그 기록 (모니터링용)
+    -->
     <Resource name="plm" auth="Container"
               type="javax.sql.DataSource" driverClassName="org.postgresql.Driver"
               url="${DB_URL}"
               username="${DB_USERNAME}" password="${DB_PASSWORD}"
-              maxTotal="200" maxIdle="50" maxWaitMillis="-1"/>
+              maxTotal="200" maxIdle="50" maxWaitMillis="10000"
+              testOnBorrow="true" validationQuery="SELECT 1"
+              removeAbandonedOnBorrow="true" removeAbandonedTimeout="60"
+              logAbandoned="true"/>
 
 </Context> 
\ No newline at end of file
-- 
2.49.1