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

[고도화] 보안 기능 고도화 - 환경변수의 사용 #84

Merged
merged 1 commit into from
Jan 19, 2023

Conversation

lmw7414
Copy link
Owner

@lmw7414 lmw7414 commented Jan 19, 2023

이 pr은 application.yaml에 노출되어 있던 각종 민감정보를 변수로 치환하여 보안을 신경쓰도록 한다.
그러나.. 사실 이미 늦었다. 이전 과거 기록(커밋 노드)을 조회하면 여전히 노출된 값을 볼 수 있기 때문.
근본적인 해결을 하려면 이 저장소를 통쨰로 지우고 새로 올리는 수 밖에 없을 것 같다.
일단 공부를 하는 의미는 있으므로 이 pr을 적용한다.

This Closes #82

보안에 취약했던 부분을 외부에서 집어넣을 수 있게끔 변경
@lmw7414 lmw7414 added the enhancement New feature or request label Jan 19, 2023
@lmw7414 lmw7414 added this to the 게시판 서비스 만들기 milestone Jan 19, 2023
@lmw7414 lmw7414 self-assigned this Jan 19, 2023
@lmw7414 lmw7414 merged commit 0fa3d8e into main Jan 19, 2023
@lmw7414 lmw7414 deleted the feature/#82-properties-for-secrets branch January 19, 2023 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[고도화] 보안 기능 고도화 - 환경변수의 사용
1 participant