Skip to content

Fix: 공통 컴포넌트 검증 및 파일 처리 안정성 개선#911

Open
KoreaNirsa wants to merge 1 commit into
eGovFramework:mainfrom
KoreaNirsa:fix/common-component-validation-hardening
Open

Fix: 공통 컴포넌트 검증 및 파일 처리 안정성 개선#911
KoreaNirsa wants to merge 1 commit into
eGovFramework:mainfrom
KoreaNirsa:fix/common-component-validation-hardening

Conversation

@KoreaNirsa
Copy link
Copy Markdown

수정 사유 Reason for modification

소스를 수정한 사유가 무엇인지 체크해 주세요. Please check the reason you modified the source. ([X] X는 대문자여야 합니다.)

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

수정된 소스 내용 Modified source

egovframework.com 공통 컴포넌트의 입력값 검증 및 파일 처리 안정성 개선

  • egovframework.com.utl.fcc.service.EgovFileUploadUtil

    • uploadFilesExt에서 파일 저장 전에 maxFileSize 제한값을 검증하도록 수정
    • 제한 크기를 초과한 파일은 저장하지 않고 SecurityException을 발생시키도록 수정
    • 파일 크기를 중복 조회하지 않도록 업로드 파일 크기 값을 재사용
  • egovframework.com.cmm.service.impl.EgovFileMngServiceImpl

    • 첨부파일 등록 결과의 atchFileId가 빈 문자열인 경우 null로 처리하도록 수정
    • 문자열 참조 비교 대신 StringUtils.isEmpty를 사용하여 빈 값 판단 안정성 개선
  • egovframework.com.cmm.web.EgovComUtlController, egovframework.com.uss.umt.web.EgovEmplyrManageController

    • /EgovPageLink.do 호출 시 linkIndex가 음수이거나 화이트리스트 범위를 벗어난 경우 공통 오류 화면으로 이동하도록 수정
    • 화이트리스트가 비어 있거나 존재하지 않는 경우도 동일하게 오류 화면으로 처리
  • egovframework.com.utl.sim.service.EgovXMLDoc

    • XML 문서 저장 처리가 정상 완료된 경우 반환값이 true가 되도록 수정
  • 테스트 코드

    • 파일 업로드 크기 제한 초과 시 저장되지 않는지 검증하는 JUnit 테스트 추가
    • 최대 허용 크기와 동일한 파일은 정상 저장되는지 검증하는 JUnit 테스트 추가
    • 공통 페이지 및 회원가입 화면 이동의 잘못된 화이트리스트 인덱스 처리 테스트 추가

JUnit 테스트 JUnit tests

테스트를 완료하셨으면 다음 항목에 [대문자X]로 표시해 주세요. When you're done testing, check the following items.

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

테스트 브라우저 Test Browser

테스트를 진행한 브라우저를 선택해 주세요. Please select the browser(s) you ran the test on. (다중 선택 가능 you can select multiple) [X] X는 대문자여야 합니다.

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

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

테스트 전과 후의 스크린샷 또는 캡처 영상을 이곳에 첨부해 주세요. Please attach screenshots or video captures of your before and after tests here.

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.

1 participant