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.

Sign up for GitHub

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

Jump to bottom

Product 경로 변수 id -> uuid 변경 #67

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Product 경로 변수 id -> uuid 변경 #67

wants to merge 3 commits into from

Conversation

JunBe
Copy link
Member

@JunBe JunBe commented Feb 5, 2025

#️⃣ Issue Number

#66

📝 작업 설명

상품 관련 엔드 포인트에서 사용하는 경로 변수인 id는 예측 가능하여 쉽게 접근할 위험이 있어 랜덤 값인 uuid를 이용하여 보안성을 높였습니다.

📸스크린샷 (선택)

💬 리뷰 요구사항

✅ PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

JunBe added 3 commits February 4, 2025 17:36
@JunBe
feat: Product 엔티티에, Response DTO uuid 컬럼 추가
767d0c1 
- 상품 관련 조회시 상품 id 값 대신 uuid를 사용하기 위해 필드 추가

- ProductResponse DTO에 id 제거, uuid 추가

Resolves: #66
@JunBe
refactor: 경로 변수 및 로직에서 id → uuid 변경
b963a35 
- ProductController: 경로 변수 id 대신 uuid 사용하도록 변경
- ProductService: 기존 id 기반 로직을 uuid로 변경

Resolves: #66
@JunBe
test: uuid 추가하여 테스트
4dc3090 
- 기존 id를 사용하여 검증하던 로직을 uuid를 사용하여 테스트 하도록 변경

Resolves: #66
@JunBe JunBe self-assigned this Feb 5, 2025
@JunBe JunBe linked an issue Feb 5, 2025 that may be closed by this pull request
productId 값을 사용하는 요청 랜덤 값 이용하도록 변경 #66
Closed
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@JunBe JunBe

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

productId 값을 사용하는 요청 랜덤 값 이용하도록 변경
1 participant
@JunBe