Freecodecamp: 잘못된 Github 인증 범위

에 만든 2016년 11월 24일 · 6코멘트 · 출처: freeCodeCamp/freeCodeCamp

github 인증 URL은 https://github.com/login?client_id=2b2a9dcc53df88ddf452&return_to=/login/oauth/authorize?client_id=2b2a9dcc53df88ddf452&redirect_uri=http://www.freecodecamp.com/auth/github/callback&response_type=code& scope=email

여기에서 scope=email 매개 변수는 https://developer.github.com/v3/oauth/#scopes 에 따라 유효한 범위가 아닙니다.

사용자 의 공개 데이터 만 검색하려는 경우 scope 는 제공 할 필요가 없습니다.
또는 공개 데이터와 비공개 이메일 ID를 모두 검색하려는 경우 scope 를 user:email 로 설정해야합니다.

현재 잘못된 scope 를 사용 하면 공개 데이터 만 검색되지만 개인 이메일 ID는 검색되지 않습니다 ( 의도적이지 않은 경우 ).
https://github.com/FreeCodeCamp/FreeCodeCamp/blob/staging/server/passport-providers.js#L147
https://github.com/FreeCodeCamp/FreeCodeCamp/blob/staging/server/passport-providers.js#L161

이것은 커밋 https://github.com/FreeCodeCamp/FreeCodeCamp/commit/2256f3ef6e2cfec7dd82b975ca5795ba792d2401 에 추가되었습니다.

help wanted

출처

abhisekp

가장 유용한 댓글

나는 그가 당신을 의미한다고 확신합니다 @abhisekp ! : sweat_smile :

raisedadead 에 2016년 11월 30일

👍2 😄1

모든 6 댓글

곧 GitHub에서 이메일 ID를 가져 오지 않고 프로필 링크 및 동기화 만 할 계획이라는 점을 고려할 때이를 공개 데이터 범위로 제한해야합니까?

예, 유효하지 않은 범위를 수정해야한다고 말했습니다.

/ cc @QuincyLarson , @BerkeleyTrue

raisedadead 에 2016년 11월 24일

@raisedadead 동의합니다. 어쨌든 GitHub를 통해 더 이상 이메일 주소가 필요하지 않으므로이 범위를 제거 할 수 있습니다.

@BerkeleyTrue 이 범위를 업데이트하면 데이터베이스에서 캠프 참가자의 이메일 주소를 잃을 위험이 있습니까?

QuincyLarson 에 2016년 11월 24일

@QuincyLarson no loopback은 이메일을 자동으로 제거하지 않습니다. 이미 가지고있는 이메일은 제거되지 않습니다.

이 범위는 안전하게 제거 할 수 있습니다.

BerkeleyTrue 에 2016년 11월 29일

@BerkeleyTrue OK- 좋아요. 확인 해주셔서 감사합니다. 이 문제를 발견 한 이후 @abhisekp 풀 리퀘스트 생성에 기여하고 싶으십니까?

QuincyLarson 에 2016년 11월 30일

😕1 😄1

나는 그가 당신을 의미한다고 확신합니다 @abhisekp ! : sweat_smile :

raisedadead 에 2016년 11월 30일

👍2 😄1

@raisedadead 예, 정정 해 주셔서 감사합니다. @abhisekp를 의미했습니다. 문제의 맨 위로 스크롤하지 못했습니다 🙂

QuincyLarson 에 2016년 11월 30일

이 페이지가 도움이 되었나요?

0 / 5 - 0 등급

Freecodecamp: 잘못된 Github 인증 범위

가장 유용한 댓글

모든 6 댓글

관련 문제