Skip to content

fix: AbstractXMLUtility 가 인스턴스마다 연 설정 컨텍스트를 닫지 않던 문제 수정, 저장 경로 직접 주입 생성자 추가 - #416

Merged
jei007 merged 1 commit into
eGovFramework:mainfrom
comghost:fix/xml-utility-context-leak
Sep 14, 2026
Merged

fix: AbstractXMLUtility 가 인스턴스마다 연 설정 컨텍스트를 닫지 않던 문제 수정, 저장 경로 직접 주입 생성자 추가#416
jei007 merged 1 commit into
eGovFramework:mainfrom
comghost:fix/xml-utility-context-leak

Conversation

@comghost

Copy link
Copy Markdown
Contributor

수정 사유 Reason for modification

  • 버그수정 Bug fixes
  • 기능개선 Enhancements
  • 기능추가 Adding features
  • 기타 Others

수정된 소스 내용 Modified source

문제

AbstractXMLUtility 의 기본 생성자는 egovxmlCfg.xml 을 읽기 위해 인스턴스마다 FileSystemXmlApplicationContext 를 새로
만든 뒤 닫지 않고 필드로 보관
합니다. 설정에서 쓰는 값은 xmlconfig 빈의 저장 경로 하나뿐인데, 컨텍스트 전체가
인스턴스 수명 동안 유지되고 close() 도 호출되지 않습니다. EgovDOMValidatorService·EgovSAXValidatorService
요청마다 만드는 코드에서는 그 횟수만큼 컨텍스트(빈 팩토리·리소스 캐시·이벤트 멀티캐스터)가 쌓입니다.

또한 설정 파일 없이는 이 유틸을 만들 수 없어, 저장 경로 하나만 필요한 상황에서도 Spring 컨텍스트와
egovxmlCfg.xml 을 준비해야 했습니다.

#268 이 같은 클래스의 저장 시 OutputStream 누수를 고쳤습니다. 이 PR 은 생성자 쪽에 남아 있던 컨텍스트 누수를 다룹니다.

수정

항목 이전 이후
기본 생성자 컨텍스트를 만들어 필드로 보관(미종료) try-with-resources 로 열어 저장 경로만 읽고 즉시 닫음
context·xmlConfig 필드 보관(다른 곳에서 미사용) 제거
저장 경로 주입 불가 protected AbstractXMLUtility(String savedPath) 신설. 두 검증기에 public 생성자 (String savedPath) 추가
new EgovDOMValidatorService() 등 기본 생성자 설정 파일에서 경로 읽음 그대로(컨텍스트만 닫음)
public AbstractXMLUtility() {
    try (FileSystemXmlApplicationContext context = new FileSystemXmlApplicationContext(CONFIG_PATH)) {
        this.savedPath = ((XmlConfig) context.getBean("xmlconfig")).getXmlpath();
    }
}

protected AbstractXMLUtility(String savedPath) {
    this.savedPath = savedPath;
}

호환성

  • 기본 생성자의 동작(설정 파일에서 저장 경로를 읽음)은 그대로이며, 컨텍스트를 닫는 것만 달라집니다. 제거한 두 필드는
    private 이고 클래스 안에서도 생성자 밖에서는 쓰이지 않았습니다.
  • 두 검증기의 기본 생성자는 손대지 않았으므로 기존 호출은 소스·바이너리 모두 그대로 동작합니다. 새 생성자는 추가일 뿐입니다.
  • savedPath 는 저장 API 가 파일명을 바로 이어 붙이는 기존 규칙을 따르므로 경로 구분자로 끝나야 하며, javadoc 에
    적었습니다(설정 값도 같은 규칙입니다).

JUnit 테스트 JUnit tests

  • JUnit 테스트 JUnit tests
  • 수동 테스트 Manual testing

AbstractXMLUtilityLifecycleTest 3건 신규, fdl.xml 모듈 전건 통과.

환경 기본 로케일(Locale) 모듈 실행 건수 결과
Windows ko_KR 6 Tests run: 6, Failures: 0, Errors: 0, Skipped: 0
Linux (Ubuntu 22.04 / OpenJDK 17) C.UTF-8 6 Tests run: 6, Failures: 0, Errors: 0, Skipped: 0
구분 건수 내용
경로 주입 2 설정 파일·컨텍스트 없이 만든 유틸이 주입 경로 아래에 XML 을 생성·저장한다 · 두 검증기의 경로 주입 생성자로 파싱까지 동작한다
기존 동작 1 기본 생성자로 만든 검증기가 종전처럼 파싱한다(기존 ControlXMLTest·XmlUtilitySaveDocumentTest 도 무변경 통과)

수동 확인: 현재 main(cc2b332) 위에서 fdl.xml 모듈 빌드·테스트가 통과함을 Windows 와 Linux 에서 확인했습니다.

테스트 브라우저 Test Browser

  • Chrome
  • Firefox
  • Edge
  • Safari
  • Opera
  • Internet Explorer
  • 기타 Others

해당 없음 — 라이브러리 내부 수정입니다.

테스트 스크린샷 또는 캡처 영상 Test screenshots or captured video

해당 없음 — 위 JUnit 테스트 결과로 갈음합니다.


base: eGovFramework:main @ cc2b332 (2026-09-10 fetch 기준, #366~#379 머지 후) · 단일 커밋

…생성자 추가

AbstractXMLUtility 의 기본 생성자는 egovxmlCfg.xml 을 읽으려고
FileSystemXmlApplicationContext 를 새로 만든 뒤 닫지 않고 필드로 보관한다. 설정에서
쓰는 값은 xmlconfig 빈의 저장 경로 하나뿐인데, 검증기(EgovDOMValidatorService·
EgovSAXValidatorService)를 만들 때마다 컨텍스트가 하나씩 남아 요청마다 유틸을
만드는 코드에서는 컨텍스트가 계속 쌓인다.

- 기본 생성자는 컨텍스트를 try-with-resources 로 열어 저장 경로만 읽고 바로 닫는다.
  보관하던 context·xmlConfig 필드는 다른 곳에서 쓰지 않으므로 제거한다
- 저장 경로를 직접 주입받는 protected 생성자를 추가하고, 두 검증기에 같은 public
  생성자를 둔다. 설정 파일과 Spring 컨텍스트 없이 쓸 수 있다. 기존 기본 생성자는
  그대로다

AbstractXMLUtilityLifecycleTest 3건 추가 — 경로 주입 생성자로 컨텍스트 없이 XML 을
생성·저장하는지, 두 검증기의 경로 주입 생성자가 파싱까지 동작하는지, 기본 생성자가
종전처럼 동작하는지.

@jei007 jei007 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

표준프레임워크에 대한 지속적인 참여에
대단히 감사드립니다.

@jei007
jei007 merged commit dbf951e into eGovFramework:main Sep 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants