Skip to content

Conversation

@parkjaehak
Copy link
Collaborator

@parkjaehak parkjaehak commented Jan 31, 2025

📄 요약(Summary)

로깅 생성 및 조회

✍🏼 상세(More)

PR Desciption

변경 사항 설명

  • LoggingAspect 클래스에 대한 로직 수정
  • API 마다 @LogType 애노테이션을 붙여주며 LogStatus에 정의된 enum 값을 적절하게 할당하면 해당 API 호출 후 로깅 생성
    @LogType(LogStatus.TASK_VIEWED)
    @Operation(summary = "작업 히스토리 조회")
    @Secured({"ROLE_MANAGER","ROLE_USER"})
    @GetMapping("/{taskId}/histories")
    public ResponseEntity<FindTaskHistoryResponse> findTaskHistories(
            @PathVariable Long taskId,
            @AuthenticationPrincipal SecurityUserDetails userInfo) {
        return ResponseEntity.ok(findTaskHistoriesUsecase.findTaskHistories(userInfo.getUserId(), taskId));
    }
  • 헥사고날 아키텍쳐 및 ddd 적용하여 로깅 관련 패키지 구조 리팩토링
  • 로깅 필터링 (로그인 기록, 작업 기록)

우선적으로 필요한 부분만 리팩토링 및 기능 구현을 진행하였습니다.
불필요코드 제거 및 로깅 관련 데이터 정제 작업에 대한 리팩토링은 필요할 것 같습니다.

Requirements for Reviewer

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

고민 중인 사항

  • customStatusCode 가 어떨때 사용되는 것인지 궁금하네요.
  • 동시성 제어를 위한 락 을 사용하기 위해 version을 사용하던데 어떻게 사용하는 것인지 확인을 해주시면 좋을 것 같습니다.
  • serverIp와 clientIp 중 조회시에 어떤 ip를 보여주는 것이 적합할까요?

첨부 자료

✅ 체크리스트(Checklist)

  • PR 양식에 맞게 작성했습니다
  • 모든 테스트가 통과했습니다
  • 프로그램이 정상적으로 작동합니다
  • 적절한 PR 라벨을 설정했습니다
  • 불필요한 코드를 제거했습니다

🚪 이슈 번호(Issue numbers)

Closes #39

@parkjaehak parkjaehak added 🧽 cleanup 코드를 더 깔끔하게 만들기만 하고, 코드 작동 방식이나 출력에 대한 부분을 변경하지 않습니다. ✨ feature 구현·개선 사항에 관련된 내용입니다 ♻️ refactor 코드가 내부적으로 작동하는 방식을 변경합니다. cleanup과는 다릅니다. HIGH 우선순위 상 labels Jan 31, 2025
@parkjaehak parkjaehak self-assigned this Jan 31, 2025
@parkjaehak parkjaehak requested a review from Sihun23 January 31, 2025 04:45
@parkjaehak parkjaehak merged commit 9cc3143 into develop Jan 31, 2025
1 check passed
@parkjaehak parkjaehak deleted the CLAP-81 branch February 3, 2025 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🧽 cleanup 코드를 더 깔끔하게 만들기만 하고, 코드 작동 방식이나 출력에 대한 부분을 변경하지 않습니다. ✨ feature 구현·개선 사항에 관련된 내용입니다 HIGH 우선순위 상 ♻️ refactor 코드가 내부적으로 작동하는 방식을 변경합니다. cleanup과는 다릅니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLAP-81 API 요청 로그 API

2 participants