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

[Docs] readme 작성 순서 변경 및 Spring Security 기능 흐름 작성 #195

Merged
merged 2 commits into from
Nov 14, 2024

Conversation

suhyeon7497
Copy link
Contributor

✨ 작업 내용

  • readme 작성 순서 변경
  • Spring Security 기능 흐름 작성

✨ 참고 사항


⏰ 현재 버그


✏ Git Close

@suhyeon7497 suhyeon7497 added the 📃 Docs 문서 작성 및 수정 (README.md 등) label Nov 14, 2024
@suhyeon7497 suhyeon7497 self-assigned this Nov 14, 2024
Comment on lines +253 to +257
- ### Spring Security
- 로그인 시, jwt로 accessToken과 refreshToken을 Cookie에 담아줍니다.
- refreshToken은 redis에 저장되어 RTR (Refresh Token Rotation)을 구현하였습니다.
- 모든 요청 AuthorizationFilter에서 Cookie에 있는 토큰이 유효한지 확인하고, 유효하면 Authenticate합니다.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수현님이 말한 흐름이 딱 이 정도 인가요?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

그냥 도메인 뭉뚱그려서 이렇게 쓸까요? API 별로 하면 너무 길어질까용

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

좀 뭉뚱 그려서 써도 될거같아요.
만약 place를 가져오는데 influence와 video가 필요하면
influencer 정보와 video 정보, place 정보를 db에서 가져온다.
그다음 response에 반환한다. 이런 느낌?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

알겠습니당

@sanghee0820 sanghee0820 merged commit 29d00e1 into weekly/11 Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📃 Docs 문서 작성 및 수정 (README.md 등)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants