rhymix icon indicating copy to clipboard operation
rhymix copied to clipboard

소셜로그인 관련 의견 취합용

Open bjrambo opened this issue 4 years ago • 8 comments

소셜로그인이 라이믹스에 추가 됨에 따라 라이믹스 자체에서 최소 조건으로 필요한 정보들이 있습니다.

일부 SNS 로그인을 지원하는 기기들은 이메일 정보 조차 넘기지 않는 경우도 있기 때문에 최소정보를 허용하는 리스트에서 이메일정도로 고정하는것이 어떠신가 합니다.

이 이슈의 목적은 위의 필수 정보 리스트중에서 최소 목적으로 하는 기능들의 의견을 포함하여 소셜로그인에서 사용될 다양한 정보들을 모아 기록 및 의견 취합을 위하여 이슈를 열었습니다.

소셜로그인에 대한 이슈들은 여기에 함께 남겨주시면 감사하겠습니다.

bjrambo avatar Sep 14 '21 06:09 bjrambo

정보가 없으면 회원가입 페이지로 넘어가 해당 빈 정보들을 입력할 수 있도록 만들 수 있는지 확인해보기 ( #1775 )

bjrambo avatar Sep 16 '21 03:09 bjrambo

소셜로그인 연동 테스트용 페이지를 만드는건 어떨까요?

현재 관리자 계정이나 소셜로그인 활성화 등 설정에는 영향을 주지 않고 API 키 등이 정상인지, 소셜로그인 서비스로부터 값을 받아올수 있는지, 서비스에서 넘겨주는 값 중 빠진 값은 없는지 확인할수 있는것도 괜찮을것 같습니다.

Lastorder-DC avatar Sep 16 '21 03:09 Lastorder-DC

혹시.. 테스트용 페이지를 라이믹스측에서 서버 부담을 해달라는 말씀이신가요??

bjrambo avatar Sep 16 '21 03:09 bjrambo

소셜로그인 API 입력후 활성화는 하지 않은 상태에서 정상 동작 여부를 알수 있도록 테스트 연동을 지원하면 어떻겠냐는 생각입니다. 그게 어떻게 테스트 서버 지원까지 가는건지 모르겠네요. 당연히 그걸 지원할 이유는 없지 않나요?

Lastorder-DC avatar Sep 16 '21 03:09 Lastorder-DC

내부적으로 소셜로그인에 대해서 준비하거나 생각 하고 있는 것이 다양한 만큼 범위를 넓게 생각하여 서버 지원이 필요하신가 하는 의문을 던졌을 뿐입니다.

최근에도 관련 부분 내용 정리를 위해 내부적으로 의견을 다룬적도 있고요.. 따라서 해당 부분에 대해서 혹시라도 의견이 타당한것인지 검토를 위해 다시 제차 제가 판단한 것이 맞는지 의견을 묻고 싶었을 뿐입니다.

그것에 대해 실망하신 모습이엿다면 죄송합니다.

다만, api입력에 대한 테스트페이지는 저도 동의하나.. 테스트를 위한 콜백 파라미터나 url설정에서 오히려 테스트를 위해서 사용되는 페이지 설정으로 좀 더 혼란을 줄수 있지 않을까 하는 생각이 들어서 고민입니다.

물론 그것을 테스트하고 변수값들을 개발자에게 전달해주면 되는데 라이믹스는 대부분은 개발자 만큼의 프로분들이 아닌 일반 사람들이 대부분이기에 혼란을 가져다줄 우려도 생각을 하고 있습니다.

bjrambo avatar Sep 16 '21 04:09 bjrambo

기분나쁘기보다는 이해가 안가서 그랬습니다. 내부에서 이루어지는 논의는 알 수 없으니까요.

다만, api입력에 대한 테스트페이지는 저도 동의하나.. 테스트를 위한 콜백 파라미터나 url설정에서 오히려 테스트를 위해서 사용되는 페이지 설정으로 좀 더 혼란을 줄수 있지 않을까 하는 생각이 들어서 고민입니다.

콜백 등은 그대로 두되, 테스트 페이지에서 출발한 요청은 연동에 사용하지 않는다던가 하는 식이면 가능하지 않을까요? (세션 변수 설정 등) 나중에 소셜 관련 문제가 생기셨다는 분에게 테스트 연동후 메세지를 알려주세요 라고만 전달드리고 나온 메세지로 원인 파악을 쉽게 할수 있을것 같아 제안드렸습니다.

Lastorder-DC avatar Sep 16 '21 04:09 Lastorder-DC

구조적으로 이미 많은 라이브러리에서 테스트 상황를 고려하지 않은체 세션까지 조작하고 있는 방식으로 얽히고 섥힌 상태이라 세션을 조작하는 것을 멈추게 한다는 작업은 만만치 않은 작업이 될 것 같네요..ㅠ 최종 디비를 입력하지 않는다면 고려해 볼 수 있으나 단순 변수나 값들을 가져오는 부분을 연동 단위에서 편의성을 줄 수 있는지 한번 최대한 검토는 해봐야할 것 같네요.

안그래도 최근에 next브랜치에 따로 커밋해둔 부분도 연동에서 문제가 생겨서 에러 메세지가 출력이 되었을때 왜 에러가 났는지에 대한 디버깅이 편하도록 할려는 개선사항도 있긴 합니다.. 그런 부분에서 최대한 신경을 우선 먼저 써본다면 추후 테스트용 페이지가 존재할 이유가 사라지지 않을까 생각도 들긴합니다.

결과론적으로 현재로써는 에러가 나타나는 경우 에러에 대한 확인이 어려운것이 사실이니 그 부분을 이런 식으로 풀어나가고 정 문제가 된다면 그 때 다시 한 번 고민해보도록 할께요.

의견 감사합니다 (_ _ )

bjrambo avatar Sep 16 '21 04:09 bjrambo

매번 소셜로그인 관련 질문을 볼때마다 API 키 확인, API 설정(반환값, 콜백) 확인, 서버 설정 확인 등 항상 대댓글로 쭉 물어봐야 하는 상황이 자주 보이곤 해서 관련해서 테스트가 쉬워지면 좋을것 같다는 생각에 제안드렸습니다. 물론 오류메세지 자체가 확인이 쉬워진다면 굳이 별도로 테스트 페이지를 둘 필요성은 없겠죠.

Lastorder-DC avatar Sep 16 '21 04:09 Lastorder-DC