Skip to content

[RELEASE] 1차 배포 - 버그 수정 사항 반영#109

Merged
hey-sung merged 2 commits intoproductionfrom
main
Apr 24, 2025
Merged

[RELEASE] 1차 배포 - 버그 수정 사항 반영#109
hey-sung merged 2 commits intoproductionfrom
main

Conversation

@hey-sung
Copy link
Collaborator

1차 배포 - 버그 수정 사항 반영

반영 브랜치

main -> production


🛠️ 개발 사항

1. 모먼트/장소의 데이터 변경 시 403 FORBIDDEN을 반환하도록 변경

  • 모먼트/장소의 데이터 변경 시, 모먼트 참여자만 가능한 권한임
  • 유효성 확인 과정에서 모먼트 참여자가 아닌 경우 403 FORBIDDEN을 반환하도록 변경 (기존에는 404 NOT FOUND 사용)

2. 커스텀 예외 핸들링이 적용되지 않는 버그 픽스

  • BusinessException이 발생했음에도 불구하고, GlobalExceptionHandler가 동작해서 커스텀 예외가 전달되지 않는 버그 발생
  • 스프링의 컴포넌트 스캔 과정에서, 클래스가 로드되는 순서가 달라져서 발생하는 문제로 판단
  • 일관된 동작 보장을 위해 @Order 어노테이션을 통해 우선순위 설정

hey-sung and others added 2 commits April 24, 2025 15:24
- BusinessException이 발생했음에도 불구하고, GlobalExceptionHandler가 동작해서 커스텀 예외가 전달되지 않는 버그 발생
- 스프링의 컴포넌트 스캔 과정에서, 클래스가 로드되는 순서가 달라져서 발생하는 문제로 판단
- 일관된 동작 보장을 위해 @order 어노테이션을 통해 우선순위 설정
- 모먼트 수정/장소 수저 요청 시, 모먼트 참여자가 아닐 때 즉, 권한이 없을 때 403 FORBIDDEN을 반환하도록 변경
@hey-sung hey-sung added 🔧 FIX 버그 픽스 📡 infra 인프라 관련 작업 labels Apr 24, 2025
@hey-sung hey-sung self-assigned this Apr 24, 2025
@github-actions
Copy link

Test Results

124 tests   124 ✅  2s ⏱️
 53 suites    0 💤
 53 files      0 ❌

Results for commit c556085.

@hey-sung hey-sung merged commit 9af4561 into production Apr 24, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔧 FIX 버그 픽스 📡 infra 인프라 관련 작업

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant