kookmin-sw / capstone-2023-08

capstone-2023-08 created by GitHub Classroom
1 stars 4 forks source link

[BE] 장고 secret key 분리 및 gitignore 업데이트 #30

Closed wynter122 closed 1 year ago

wynter122 commented 1 year ago

장고의 secret key를 수동으로 추가하는것이 번거로워 환경변수 접근 방식으로 변경했습니다. 또한, DB 파일도 숨김처리했습니다.

secrets.json 파일 혹은 secret key 값이 필요하시면 카톡방에서 말씀해주세요 manage.py 와 같은 위치에 두고 사용하시면 됩니다.

wynter122 commented 1 year ago

@dh5473 방법 괜찮으면 리뷰 어프루브 해주시면 감사하겠습니다! 지금 Pr이 약간 꼬여서 제가 올린 PR을 좀 정리해야 할 것 같아서요!!

dh5473 commented 1 year ago

어..server 공간에 json파일 올려두고 json파일에 저장된 secret 키를 읽는 방식인가요? 환경 변수 설정은 다른 방법이 있긴 해요! 그리고 지금 settings.py 내에 with open을 비롯하여 함수가 들어갔는데 좋은 방식은 아닌 것 같아요. 만약 이 방식을 쓰고 싶다하면 utils.py 같은 파일 따로 만들어서 import하는 방식으로 불러오든가 해야하지 않을까요?

dh5473 commented 1 year ago

맥 os는 어떨지 모르겠는데, 아마 bash 파일에 export 변수명=시크릿키 저장하시고 장고 settings.py에서 SECRET_KEY = os.environ['변수명'] 으로 바꿔두시면 바로 접속될 거에요.

지금 짜신 것은 사실 환경 변수라기보다 json 파일로 따로 저장해두고 불러오는 건데, 보안 혹은 클린 코드 측면에서 이 방법을 사용하는 게 좋아보여요. 환경 변수 저장 방법은 구글링하면 금방 나올 거에요!

wynter122 commented 1 year ago

오! 그 방법 있는걸 완전 잊어버리고있었네요 피드백 감사합니다!! 이 PR은 삭제하고 노트북 환경변수 지정으로 개발하고, 나중에 EC2에서도 그렇게 개발 이어나가면 완전 깔끔할 것 같아요!! 감사합니다👍