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/user/issue2 #12

Closed
wants to merge 13 commits into from
Closed

Feat/user/issue2 #12

wants to merge 13 commits into from

Conversation

smellHyang
Copy link
Contributor

@smellHyang smellHyang commented Jul 8, 2022 •
edited
Loading

기능 구현

  • 일반 로그인, 회원가입
  • Oauth 로그인, 회원가입(google)

리팩토링 필요

  • mustache사용으로 임시적 client 구현
  • mustache사용으로 @RestController사용 불가하여 임시적 @controller로 구성
  • 회원가입 controller, service 분리 필요
  • 로그인 및 회원가입 기능 유효성 검증 추가
  • IndexController분리 필요
  • 주석 및 Import 정리 필요
  • cloud db 연결 필요

논의사항

  • 권한 분리 기능
  • google 외 oauth 추가(collection - provider 추가 필요)

dayoungMM and others added 4 commits July 6, 2022 00:24
@dayoungMM
refact: config server에서 응답을 받지 못해도 구동 되도록 수정
48a4c0b 
- config 정보가 없어 스프링 컨테이너가 뜨지 못하는 상황은 발생할 수 있으나, config server가 응답을 주지 않는다는 이유로 애플리케이션이 구동되지 않는 상황은 발생하지 않도록
@dayoungMM
feat: DB 연결
3dc2a48 
- local 에 존재하는 MongoDB
- resources/application-db.properties 파일 필요 (로그인 정보 포함), github에는 올리지 않을 예정
@dayoungMM
feat: OAuth2 적용
6545187 
- spring-boot-starter-oauth2-client dependency를 추가하면 spring-boot-starter-security도 추가해야함
- 안그러면 controller에 request가 가지 않음
- resources에 application-oauth.properties 있어야 동작
@smellHyang
feat/config/issue2
df342f8 
- 일반 로그인, 회원가입
- oauth 로그인, 회원가입 기능 구현
@smellHyang smellHyang added the devsta-users [MSA] user, chatting service label Jul 8, 2022
@smellHyang smellHyang requested a review from dayoungMM July 8, 2022 10:04
This was linked to issues Jul 8, 2022
[user] 회원가입 #10
Closed
[user-1] 로그인 기능 #8
Open
@dayoungMM
Copy link
Contributor

dayoungMM commented Jul 9, 2022 •
edited
Loading

1. mustache사용으로 @RestController사용 불가하여 임시적 @controller로 구성에 대한 보충설명

  • UserController는 HTTP 통신하는 API 입니다.
  • mustache는 web 페이지를 보여주는 것이기 때문에 새로운 controller가 필요한 것 같아 이후 작업에는 따로 controller 분리해서 사용하겠습니다.

2. commit 파일

  • 민감정보가 있는 파일은 커밋하지 말아주세요
  • .idea 파일이나 library 파일은 커밋하지 않으셔도 됩니다!

@dayoungMM
Copy link
Contributor

코드 수정하여 다시 PR 하겠습니다

@dayoungMM dayoungMM closed this Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dayoungMM dayoungMM Awaiting requested review from dayoungMM

Assignees
No one assigned
Labels
devsta-users [MSA] user, chatting service
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[user-1] 로그인 기능 [user] 회원가입
2 participants
@smellHyang @dayoungMM