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.

Sign up for GitHub

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

Jump to bottom

feat: [#131] Security 적용 #139

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

feat: [#131] Security 적용 #139

wants to merge 1 commit into from

Conversation

kuk6933
Copy link
Contributor

@kuk6933 kuk6933 commented Sep 28, 2023

작업 내용

  • API에 Security를 적용했습니다

상세 내용

  • login, 회원 생성, 닉네임 체크, 비밀번호 초기화 등 requestMatchers에 있는 url들 이외에는 Token을 넣어야 동작합니다
  • enums, apns, ws등 개발이 진행중인 것들에 대해서는 아직 적용하지 않았으나 말씀 주시면 적용하겠습니다
  • 개발을 위해 Acceess Token 기한을 늘렸기에 merge된 후 공유해드리는 토큰을 70일동안은 사용하실 수 있습니다

관련 이슈

@kuk6933 kuk6933 added the 🛠️ 구현 개선 구현 개선 사항에 관련된 내용입니다. label Sep 28, 2023
@kuk6933 kuk6933 self-assigned this Sep 28, 2023
jaewonLeeKOR
jaewonLeeKOR approved these changes Nov 3, 2023
View reviewed changes
Copy link
Member

@jaewonLeeKOR jaewonLeeKOR left a comment

Choose a reason for hiding this comment

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

채팅은 필터에서 뺀거 맞죠?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jaewonLeeKOR jaewonLeeKOR jaewonLeeKOR approved these changes

@lreowy lreowy Awaiting requested review from lreowy

@minjgziii minjgziii Awaiting requested review from minjgziii

At least 3 approving reviews are required to merge this pull request.

Assignees

@kuk6933 kuk6933

Labels
🛠️ 구현 개선 구현 개선 사항에 관련된 내용입니다.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kuk6933 @jaewonLeeKOR