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

[week2] 블로그 작성 #14

Merged
merged 1 commit into from
Nov 16, 2024
Merged

[week2] 블로그 작성 #14

merged 1 commit into from
Nov 16, 2024

Conversation

parkje0927
Copy link
Contributor

OAuth2.0 Microsoft 소셜 로그인을 주제로 작성했습니다.

소셜 로그인 중에 레퍼런스도 많이 없고 연동하면서 Spring Security 흐름에 대해 더 공부하게 된 계기가 되었던 주제라서 글로 작성해보았습니다.
생략한 부분이나 부족한 부분들에 대해 피드백 주시면 확인하도록 하겠습니다!

Copy link

github-actions bot commented Nov 7, 2024

Chat GPT's review

Blog Link: https://jjungdev.tistory.com/38
블로그 제목: OAuth2.0 Microsoft 소셜 로그인

요약: 이 블로그 글은 소셜 로그인을 연동하면서 Microsoft 소셜 로그인을 처음 시도한 경험을 공유합니다. Microsoft 로그인을 위한 앱 설정부터 Spring Security 설정, OAuth2.0 흐름과 custom 설계에 이르기까지 다양한 설정과 과정을 다루고 있습니다.

피드백:

  1. 글의 목적: 글의 목적이 명확하게 전달되었습니다. 작성자가 Microsoft 소셜 로그인을 경험하며 어떤 과정을 거쳤는지를 상세하게 공유하고 있습니다.

  2. 구조와 전개: 글은 앱 설정, Spring Security 설정, OAuth2.0 흐름과 custom 설계로 구조화되어 있어 내용을 논리적으로 전개하고 있습니다. 각 단계를 순서대로 설명하여 이해하기 쉽게 되어 있습니다.

  3. 명확성과 가독성: 전반적으로 글은 명확하고 가독성이 좋습니다. 그러나 코드 부분이 많아 글이 조금 밀집하게 느껴질 수 있습니다. 코드를 더 간결하게 표현하거나 주요 내용을 요약하여 강조할 수 있을 것입니다.

  4. 어조 및 스타일: 글의 어조는 친절하고 전문적입니다. 작성자의 경험을 공유하고 있어 독자들과 소통하는 느낌을 주고 있습니다.

  5. 개선 사항 및 총평: 글의 내용과 전달하려는 정보는 훌륭하나, 코드 부분이 조금 지루하게 느껴질 수 있습니다. 개념을 잘 이해하고 싶어하는 독자에게 도움을 줄 수 있는 추가 설명이 필요할 것으로 보입니다. 전반적으로 좋은 글이지만 가급적이면 간결한 코드와 요약된 내용을 추가하면 독자들에게 더 쉽게 전달될 것입니다.

@nowgnas
Copy link
Member

nowgnas commented Nov 12, 2024

정현님! MS 소셜 로그인 너무 흥미롭게 봤습니다!!
이전에 프로젝트를 하면서 소셜 로그인을 많이 했어서 더 관심있게 읽었던거 같네요!

다른 소셜 로그인 연결에는 tenant라는 개념이 없어서 처음 오류를 마주쳤을 때 어려웠을 것 같은데 어떤 개념인지 설명을 적어 두어서 MS 소셜 로그인을 구현하려는 사람들한테 도움이 많이 될거 같습니다!

OAuth 2.0 시퀀스 다이어그램으로 글과 함께 봤을 때 이해하기 좋았던 것 같아요!!
또 보통 여러개의 소셜 로그인을 사용하는 경우가 많은데 그에 따른 custom 설계를 코드와 함게 설명되어 있어서 흐름 따라가기 좋았던 것 같습니다.

MS 소셜 로그인 도입에 대한 글이 거의 없었는데 정현님의 글이 다른 개발자들에게 도움이 많이 될 것 같아 좋네요!!
어려운 내용 잘 정리해 주신 것 같고 고생 하셨습니다!!

@nowgnas nowgnas merged commit 55897fe into main Nov 16, 2024
1 check passed
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.

2 participants