🐛 Fix: Party 수정, 삭제 권한 확인 로직 수정 - [merged] #69

Closed GATBAWI closed 5 months ago

In GitLab by @soomni20 on May 15, 2024, 17:44

Merges BE/fix/party-update -> develop

📄 제목

✍ 작업 내용 [FE/BE]

작성자인지 확인하는 로직 수정

MR 타입

[ ] 기능 추가
[x] 기능 수정
[ ] 기능 삭제
[ ] 버그 수정
[ ] 코드 리팩토링
[ ] UI 추가 및 변경

반영 브랜치

fix/party-update-> develop

테스트 결과

[x] Swagger 테스트 결과 이상 없습니다.
[ ] local ci 테스트 결과 이상 없습니다.
[ ] Postman 테스트 결과 이상 없습니다.

In GitLab by @oistmil on May 15, 2024, 17:47

확장성이 좀 더 고려되었네요

In GitLab by @oistmil on May 15, 2024, 17:47

approved this merge request

In GitLab by @yaj518 on May 15, 2024, 17:48

approved this merge request

In GitLab by @sgbae777 on May 16, 2024, 08:46

Commented on 5d7c23e7c2c5ad0ba177aaa8eb80b0a552b7061c

이렇게 바꾼 이유가 궁금합니다

In GitLab by @sgbae777 on May 16, 2024, 08:46

approved this merge request

In GitLab by @soomni20 on May 16, 2024, 09:18

Commented on 5d7c23e7c2c5ad0ba177aaa8eb80b0a552b7061c

자동 매칭 때문입니다. 자동 매칭은 웹 소켓 통신으로 매칭이 이루어지면 파티가 생성되도록 했습니다. http 통신이 아니기 때문에 SecurityContexxtHolder에 인증 주체가 담겨져 있지 않습니다. 다른 api 요청들은 jwt 토큰으로 사용자를 검증하고 검증한 뒤에 사용자의 정보를 SecurityContextHolder에 담게됩니다. (아래 그림 참고) 그런데 웹 소켓 통신은 다른 인증 방식을 사용하기 때문에 사용자 정보가 담겨져 있지 않고 그래서 BaseBy Entity의 createdBy에 값이 들어가지 않습니다. 그래서 기존 방식으로 할 경우에 비교 대상인 createBy가 항상 null이므로 검증이 불가합니다.

보안 담당자 여아정씨께서 웹소켓 인증 방식을 적용 중이십니다.

In GitLab by @sgbae777 on May 16, 2024, 09:21

Commented on 5d7c23e7c2c5ad0ba177aaa8eb80b0a552b7061c

소켓 연결할때, pub,sub 할때 토큰을 담아서 하는데 인증 방식이 달라서 토큰으로 사용자 정보를 알 수 없다는 말인가요?

In GitLab by @soomni20 on May 16, 2024, 09:22

Commented on 5d7c23e7c2c5ad0ba177aaa8eb80b0a552b7061c

예이 알아본 봐로는 그렇습니다. 하지만 배움이 부족한 상태이므로 특렸을수도... 그래서 리팩토링 하면서 공부해보려고 합니당 우핫

In GitLab by @sgbae777 on May 16, 2024, 09:31

Commented on 5d7c23e7c2c5ad0ba177aaa8eb80b0a552b7061c

예 수고 많으셨습니다

In GitLab by @sgbae777 on May 16, 2024, 09:31

resolved all threads