Skip to content

Conversation

@Jjiggu
Copy link
Contributor

@Jjiggu Jjiggu commented Dec 23, 2025

작업 요약

  • securityconfig 내 authorizeHttpRequests 경로 추가

Issue Link

#338
#340

문제점 및 어려움

해결 방안

Reference

Summary by CodeRabbit

  • 보안
    • 주문 기능의 보안 설정이 개선되었습니다.
    • 일부 기존 접근 경로가 업데이트되었습니다.

✏️ Tip: You can customize this high-level summary in your review settings.

@Jjiggu Jjiggu self-assigned this Dec 23, 2025
@Jjiggu Jjiggu added the bug Something isn't working label Dec 23, 2025
@Jjiggu Jjiggu merged commit 8f87261 into develop Dec 23, 2025
1 check passed
@coderabbitai
Copy link

coderabbitai bot commented Dec 23, 2025

Caution

Review failed

The pull request is closed.

개요

보안 설정 파일의 요청 경로 매처를 업데이트했습니다. /v1/stores/*/tables/*/orders 경로에 대한 새로운 보안 라우트 매처를 추가하고, /orders/**/v1/menus/all-menus/stores/** 기존 경로 매처를 제거했습니다.

변경 사항

집단 / 파일 변경 요약
보안 설정
nowait-app-user-api/src/main/java/com/nowait/applicationuser/config/security/SecurityConfig.java
테이블 주문 생성 및 조회를 위한 새로운 경로 매처 /v1/stores/*/tables/*/orders 추가. 기존 /orders/**/v1/menus/all-menus/stores/** 경로 매처 제거.

예상 코드 리뷰 시간

🎯 2 (Simple) | ⏱️ ~5-10 minutes

관련 이슈

관련 PR

제안 리뷰어

  • HyemIin
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/#340-order-auth-bug-fix

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between fdb2a17 and b7d65f9.

📒 Files selected for processing (1)
  • nowait-app-user-api/src/main/java/com/nowait/applicationuser/config/security/SecurityConfig.java

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions bot requested a review from HyemIin December 23, 2025 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants