5월 백엔드팀 블로그 #285
Merged
5월 백엔드팀 블로그 #285
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
iju1633
requested changes
May 25, 2023
_posts/2023-05-02-jwt.md
Outdated
Comment on lines
18
to
19
| jwt는 이를 달성하기 위한 하나의 도구입니다. | ||
| 이 용도로 사용되는 것에는 JWT 외에도 쿠키와 세션이 있지만 이 글에서는 우리의 관심사인 JWT 토큰에 대해서 다뤄보도록 하겠습니다. |
There was a problem hiding this comment.
블로그 내용에서 코드를 제외하고 설명할 때의 jwt 표현을 통일하면 더 좋을 것 같습니다 ! (jwt or JWT)
There was a problem hiding this comment.
저도 똑같이 생각했습니다! 수정하면 더 보기 좋을 것 같습니다.
There was a problem hiding this comment.
앗 제목과 소제목은 대문자로 하고 내용은 소문자로 하려 했는데 놓친 부분이 있었군요ㅠㅠ
말씀해주셔서 감사합니당! 수정하는 김에 통일성 있게 다 소문자로 수정했습니다!
_posts/2023-05-02-jwt.md
Outdated
|
|
||
| ## JWT의 약한 보안성 보완법 | ||
|
|
||
| jwt는 토큰을 서버에서 관리하지 않기에 발급한 토큰을 무효화할 수 없다는 단점이 있습니다. |
There was a problem hiding this comment.
jwt -> jwt를 통한 사용자 관리는 과 같이 추가적인 설명이 필요해보입니다.
_posts/2023-05-02-jwt.md
Outdated
| 페이로드는 key-value의 형태로 사용자가 원하는 정보들을 담을 수 있습니다. | ||
| 마지막으로 시그니쳐에 대해 살펴보기 전에 알아야 할 부분이 있습니다. | ||
| 헤더와 페이로드에 담긴 정보는 누구든지 볼 수 있기에 민감한 정보를 담아서는 안된다는 것입니다. | ||
| (따라서 위의 jwt 예시에서도 페이로드에 jwt의 생성, 만료시간 외에 유저의 인덱스 정보밖에 없는걸 볼 수 있습니다.) |
There was a problem hiding this comment.
[Nit] 따라서 위의 사진에서도 페이로드 부분에는 생성, 만료시간 외에 유저의 인덱스 정보만 있는 것을 확인할 수 있습니다. 등 과 같이 표현을 수정한다면 가독성이 좀 더 좋을 것 같습니다.
pasly0920
previously approved these changes
May 26, 2023
Comment on lines
+49
to
+50
| 시그니처를 구성하는 값 중에서 핵심은 시크릿 키입니다. 나머지 정보는 외부의 사용자가 모두 확인 가능하지만 시크릿 키는 아니기 때문입니다. | ||
| 그래서 서비스 제공자는 보관을 잘 할 필요가 있으며 학생 개발자들의 경우 실수로라도 시크릿 키를 깃허브 등의 오픈 플랫폼에 업로드해서는 안 됩니다. |
There was a problem hiding this comment.
시크릿 키를 어떻게 보관하는 지에 대한 예시가 있으면 더 좋을 것 같습니다.
iju1633
added
✅ 리뷰 완료
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.