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

Refactor: UNVALIDATED 에서 GUEST 로 권한을 변경하고 가입시 기존 권한을 CREW 에서 GUEST 로 변경한다. #1568

Merged
merged 50 commits into from
Oct 4, 2023

Conversation

splitCoding
Copy link
Contributor

@splitCoding splitCoding commented Oct 4, 2023

#️⃣연관된 이슈

📝작업 내용

1. 기존에 존재하던 Role 의 UNVALIDATEDGUEST 로 변경한다.

2. GIthubLogin 시 부여되는 권한을 CREW 에서 GUEST 로 변경한다.

  • Role 에 권한에 중요도 (importance) 를 부여한다.
    • GUEST : 0
    • CREW : 1
    • COACH : 2
    • ADMIN : 3

3. 회원의 권한을 변경하는 기능을 구현한다.

  • API
    • method : PATCH
    • Authorization Header
    • url : /member/{member_id}/role
  • 과정
    • 변경을 요청하는 멤버의 토큰에서 Role 을 검증한다. ( 권한 변경을 가능한 멤버를 manager 라고 칭하겠습니다. )
    • 요청한 멤버가 manager 권한 이상의 권한을 가진 멤버라면 member_id 에 해당하는 멤버의 권한을 변경하도록 한다.
  • 속성 추가
    • manager.roleapplication.yml 속성에 추가한다. ( 환경 별 QA, 테스트를 위해 profile 별 속성을 다르게 설정한다. )
    • profile 별 manager.role
      • prod : Role.COACH
      • dev : Role.GUEST ( DB에 직접 접근하여 권한을 바꾸고 나서 QA를 진행하지 않아도 되도록 GUEST 로 설정 )
      • local : Role.GUEST( DB에 직접 접근하여 권한을 바꾸고 나서 QA를 진행하지 않아도 되도록 GUEST 로 설정 )
      • test : Role.GUEST( DB에 직접 접근하여 권한을 바꾸고 나서 QA를 진행하지 않아도 되도록 GUEST 로 설정 )

etc

  • CREW 권한이 필요한 아티클 기능에 검증로직을 추가한다.
  • 추가적으로 바뀐 로직으로 통해 test 모두 수정

QA 결과

  • 초기 회원가입시 GUEST 권한 지정 확인 완료 ( dev, prod 모두 확인 완료 )
image
  • dev 환경 멤버 권한 변경 확인 완료 ( dev 환경 완료, prod 환경 캠퍼스에서 진행 예정 )
image
  • GUEST 권한시 아티클 글 작성 불가 확인 완료 ( dev, prod 모두 확인 완료 )
image

Creative-Lee and others added 30 commits July 26, 2023 17:17
…ctor

Refactor/#1426 아티클 첫번째 스프린트 마무리를 위한 중간 merge
…riting

Hotfix/1451 프롤로그 글쓰기 기능 핫픽스
…ctor

Refactor/#1465 아티클 네비게이션 탭 활성화 및 링크 이동기능 추가
…ctor

Refactor/#1426 meta og 관련 기능 추가 및 아티클 로직 개선
…ctor

refactor/#1482 변경된 metaOg api 반영 및 타입 추가
meta og 태그를 이용하여 아티클 기능 구현
아티클 MetaOG 빈문자열 관련 서버오류 처리 및 클라이언트 기본값 바인딩 기능 오류 수정
…eButton

refactor: Meta Og 글쓰기 버튼 로그인 시에만 활성화 되도록 변경
refactor: 아티클 생성시 CREW 권한 확인 로직 추가
[FEAT] 아티클 작성시 크루 권한 확인
…-react-router-dom

react router dom 유틸 훅 추가
남은 components 폴더의 js 파일을 ts로 마이그레이션한다
feb-dain and others added 20 commits September 4, 2023 15:37
refactor : ts로 마이그레이션한다
fix: 타입 검사 건너뛰는 코드 제거
feat: js ts 마이그레이션 반영
refactor: js ts 마이그레이션
@splitCoding splitCoding self-assigned this Oct 4, 2023
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 4, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

93.8% 93.8% Coverage
0.0% 0.0% Duplication

warning The version of Java (11.0.20) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

@splitCoding splitCoding changed the title Refactor/guest 권한 Refactor: UNVALIDATED 에서 GUEST 로 권한을 변경하고 가입시 기존 권한을 CREW 에서 GUEST 로 변경한다. Oct 4, 2023
@splitCoding splitCoding merged commit d3343e9 into main Oct 4, 2023
4 checks 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.

GithubLogin 을 통해 가입되는 유저 기본권한을 CREW 에서 GUEST 로 변경한다.
9 participants