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

fix : jasypt 비밀번호 변경으로 인한 재 암호화 #743

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

hyeon0208
Copy link
Contributor

🚩 연관 이슈

close #740


📝 작업 내용

PR에 jasypt 비밀번호가 이미지로 노출되어 재 암호화 진행했어요 😭
복호화한 파일은 노션 -> BE 애플리케이션 설정 관리 에 정리해두었습니다
변경된 비밀번호는 노션 -> 계정에 작성해두었어요

참고로 이 PR을 머지할 때 Git Action의 jasypt 비밀번호 환경변수를 변경해야해서
merge는 제가 하겠습니다 🙇🏼‍♂️


🏞️ 스크린샷 (선택)


🗣️ 리뷰 요구사항 (선택)

Copy link

Test Results

156 tests   156 ✅  5s ⏱️
 45 suites    0 💤
 45 files      0 ❌

Results for commit e063e8b.

Copy link

📝 Test Coverage Report

Overall Project 80.02% 🍏

There is no coverage information present for the Files changed

Copy link
Contributor

@eun-byeol eun-byeol left a comment

Choose a reason for hiding this comment

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

🔒🔒🔒

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fix: Jaspty 암호 변경
2 participants