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

Remove ID and secret from code #17

Closed
wants to merge 1 commit into from

Conversation

marocchino
Copy link

@marocchino marocchino commented Nov 22, 2016

close #6

로컬에서는 루트에 .env 만드시고

NAVER_MAP_CLIENT_ID=xxx
NAVER_MAP_CLIENT_SECRET=yyy

넣으시면 이전과 같이 동작합니다.

트라비스에서는 컨피그 눌러서 환경변수에 넣으시면 됩니다.
https://docs.travis-ci.com/user/environment-variables/#Defining-Variables-in-Repository-Settings

제 커밋 없어져도 상관없으니 가능하면 이력에서 민감한 정보 지우세요.

이링크가 도움이 될겁니다.
https://help.github.com/articles/remove-sensitive-data/

@marocchino
Copy link
Author

생각해보니 그냥 시크릿 키만 재발급할 수도 있겠네요. -3-

@adhrinae
Copy link
Contributor

저도 처음에 보고 잠깐 어? 했다가 아무렇지도 않게 계속 작업을 하고 있었네요
'어차피 지도 API정도 괜찮겠지' 하구요
@marocchino 님이 제시해주신대로 관리하는게 맞다고 생각합니다

@say8425
Copy link
Owner

say8425 commented Nov 23, 2016

아,, 사실은 저도 travis에 환경변수 세팅이 있다는 것을 발견했기는 했는데
계속 미루고 있었군요 소스 잇다가 리뷰할게요

@say8425
Copy link
Owner

say8425 commented Nov 23, 2016

@marocchino 혹시 bundle install 해주셔서 다시올려주실 수 있으시나요? gem이 설치가 안되서 동작을 안하네요ㅠ

@marocchino
Copy link
Author

@say8425
Copy link
Owner

say8425 commented Nov 23, 2016

환경변수로 분리하는 작업이 상당히 곤란해졌습니다.
우선 말씀하신 travis ci에 환경변수를 제공하는 방법을 쓰면 fork로 따온 풀리는 환경변수를 제공받지 못합니다.
그리고 키가 없다는 것은 테스트 코드를 전혀 동작시킬 수 없다는 것을 의미하고요.

Encrypted environment variables are not available to pull requests from forks due to the security risk of exposing such information to unknown code.

정확히는 travis는 fork로 따온 풀리에는 암호화된 환경변수를 보안 이유로 제공하지 않습니다.

따라서 쓸 수 있는 방법은 아예 대놓고 환경변수를 gem에 플레인으로 포함시키는 건데,
이러면 test 코드에 키를 하드코딩 한 것과 차이가 없어집니다.
또한 찾아보니 네이버측 약관에도 위배될 가능성이 있습니다.

다만 포크 온 것도 환경변수를 제공하게 만드는 것처럼 보이는 방법을 찾기는 했으나
일단 트라비스 측에서 보안상 기본적으로 제공안하는 기능이라서 이 방법을 써도될지 곤란해졌습니다.

@marocchino
Copy link
Author

그럼 체리픽해서 마스터에 직접 넣으세요.

@marocchino
Copy link
Author

#19 랑 중복되니 닫겠습니다.

@marocchino marocchino closed this Dec 3, 2016
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.

client key와 secrt key를 config에서 저장 받아 쓰는 기능 구현
3 participants