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

[BE] 사용되지 않는 기능 제거 및 권한 에러 수정 #31

Merged
merged 3 commits into from
Aug 24, 2024

Conversation

seungh1024
Copy link
Collaborator

@seungh1024 seungh1024 commented Aug 24, 2024

  • resolves #

🚀 어떤 기능을 구현했나요 ?

  • 이전 기능의 권한 인증과 AOP 권한 인증의 충돌로 발생한 에러 수정
  • 파일의 소유자도 읽기 권한에 대한 폴더에 쓰기 작업을 할 수 없었던 에러 수정

🔥 어떻게 해결했나요 ?

  • 기존 권한 인증 제거
  • 읽기 권한에 쓰기 작업 요청 시 본인의 폴더인지 추가로 확인

📝 어떤 부분에 집중해서 리뷰해야 할까요?

📚 참고 자료, 할 말

- 파일 이동 시 소유자 확인하는 코드 제거(AOP로 대체됨)
- 기본 expired 시간 수정
- 로그 추가
@seungh1024 seungh1024 self-assigned this Aug 24, 2024
- 모니터링을 위한 프로메테우스 추가
- 자신의 파일이 읽기 권한일 때 하위 파일 생성과 이동이 불가능 했던 에러 수정
@seungh1024 seungh1024 changed the title [BE] 사용되지 않는 기능 제거 [BE] 사용되지 않는 기능 제거 및 권한 에러 수정 Aug 24, 2024
@seungh1024 seungh1024 requested a review from i960107 August 24, 2024 07:56
@seungh1024 seungh1024 added the 🐞버그 해결 Something isn't working label Aug 24, 2024
@i960107 i960107 merged commit ddd97fa into main Aug 24, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞버그 해결 Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants