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

url safe 를 고려한 암복호화 로직 개선 #157

Open
wants to merge 2 commits into
base: contribution
Choose a base branch
from

Conversation

kiboomhan
Copy link

수정 사유 Reason for modification

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

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

수정된 소스 내용 Modified source

이슈

  • 기존 url safe 를 고려한 암복호화 로직 개선 필요성
  • URLEncoder 를 통해 나온 결과는 url 사용을 위함이고, 이 자체가 url safe 하지 못함(특수문자 존재)
  • 복잡한 케이스 고려 필요(SSR 케이스, 스프링 자동 URL디코딩 케이스, 정상 케이스 등)

처리내용

  • URLEncoder 제거 및 Base64 url safe 옵션 활성화
  • 불필요한 인스턴스 생성 제거

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.

  • 테스트 전
2024-08-22 11:10:08,811  INFO [org.egovframe.rte.fdl.cryptography.EgovARIAErrorTest]  원본(orignal):ckimage/2018/12
2024-08-22 11:10:08,812  INFO [org.egovframe.rte.fdl.cryptography.EgovARIAErrorTest]  인코딩(encrypted):%2FITkIPGdXOGtfGQ7KO5n1A%3D%3D
2024-08-22 11:10:08,812  INFO [org.egovframe.rte.fdl.cryptography.EgovARIAErrorTest]  디코딩(decrypted):ckimage/2018/12
2024-08-22 11:10:08,812  INFO [org.egovframe.rte.fdl.cryptography.EgovARIAErrorTest]  URL인코딩(encrypted+url):%252FITkIPGdXOGtfGQ7KO5n1A%253D%253D
2024-08-22 11:10:08,812 ERROR [org.egovframe.rte.fdl.cryptography.EgovARIAErrorTest]  실패 !!!(URL SAFE 실패)
  • 테스트 후
2024-08-22 11:08:20,756  INFO [org.egovframe.rte.fdl.cryptography.EgovARIAErrorTest]  원본(orignal):ckimage/2018/12
2024-08-22 11:08:20,756  INFO [org.egovframe.rte.fdl.cryptography.EgovARIAErrorTest]  인코딩(encrypted):_ITkIPGdXOGtfGQ7KO5n1A
2024-08-22 11:08:20,757  INFO [org.egovframe.rte.fdl.cryptography.EgovARIAErrorTest]  디코딩(decrypted):ckimage/2018/12
2024-08-22 11:08:20,757  INFO [org.egovframe.rte.fdl.cryptography.EgovARIAErrorTest]  URL인코딩(encrypted+url):_ITkIPGdXOGtfGQ7KO5n1A
2024-08-22 11:08:20,757  INFO [org.egovframe.rte.fdl.cryptography.EgovARIAErrorTest]  통과 !!!

@kiboomhan kiboomhan changed the title Feature/improve encoding decoding url safe 를 고려한 암복호화 로직 개선 Aug 22, 2024
@eGovFrameSupport
Copy link
Contributor

안녕하세요
전자정부 표준프레임워크 센터입니다.

'24년 7월 29일부터 9월 30일까지 전자정부 서비스 개발 표준 기반인 "전자정부 표준프레임워크" 개선에 민간 SW개발자의 아이디어와 의견을 반영하기 위해 "2024년 전자정부 표준프레임워크 컨트리뷰션"을 개최 하였습니다.

표준프레임워크 깃허브(github.com/egovframework)에 보내주신 귀하의 의견에 감사드리며, 전자정부 표준프레임워크 운영·개선 과정에서 유용하게 활용토록 하겠습니다.
본 설문에 응답 주시면 표준프레임워크에서 마련한 소정의 기념품을 발송 드릴 예정이오니, 문항에 빠짐없이 응답 해 주시기 바랍니다.

앞으로도 전자정부 표준프레임워크에 지속적인 관심 바라오며, 아래 URL 접속, 설문지로 이동하여 항목에 응답 부탁드립니다.
https://forms.gle/6qRXGFZn5WTaxBCK8

※ 컨트리뷰션 종료 후, 시상관련하여 연락 갈 예정이오니, 꼭 통화 가능한 본인번호를 기재해 주시기 바랍니다.
※ 귀하께서 답변해 주시는 내용은 컨트리뷰션 참가 확인 및 기념품 발송 목적으로만 활용되며, 개인정보는 기념품 발송 후 즉시 영구 삭제처리 됩니다.

감사합니다.

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