SPMS/SPMS_API
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: JWT 토큰에 adminId 클레임 추가 (#260) #261

Merged
seonkyu.kim merged 1 commits from fix/#260-jwt-adminid-claim into develop 2026-02-28 09:37:18 +00:00
Conversation 0 Commits 1 Files Changed 1 +2 -1
seonkyu.kim commented 2026-02-28 09:35:43 +00:00
Owner
Copy Link

📋 작업 요약

  • JWT 토큰 생성 시 adminId 클레임이 누락되어 모든 컨트롤러에서 500 에러 발생하는 버그 수정

🔗 관련 이슈 (Related Issues)

Closes #260

🛠️ 작업 내용 (Changes)

  • JwtService.cs — GenerateAccessToken에 "adminId" 클레임 추가

📢 리뷰어 참고 사항 (To Reviewers)

  • 기존 토큰에는 adminId 클레임이 없으므로 반드시 재로그인 필요
  • sub 클레임은 JWT 표준으로 유지, adminId는 비즈니스 로직용으로 별도 추가

✅ 체크리스트 (Self Checklist)

  • 빌드(Build)가 성공적으로 수행되었는가?
  • 모든 단위 테스트(Unit Test)를 통과하였는가?
  • 불필요한 로그나 주석을 제거하였는가?
  • 컨벤션(Clean Architecture, Naming)을 준수하였는가?
  • 기밀 정보(비밀번호, 키 등)가 하드코딩 되어있지 않은가?

📸 스크린샷 / 테스트 로그 (Screenshots/Logs)

빌드 성공: 경고 0개, 오류 0개

## 📋 작업 요약 - JWT 토큰 생성 시 `adminId` 클레임이 누락되어 모든 컨트롤러에서 500 에러 발생하는 버그 수정 ## 🔗 관련 이슈 (Related Issues) Closes #260 ## 🛠️ 작업 내용 (Changes) - [x] `JwtService.cs` — `GenerateAccessToken`에 `"adminId"` 클레임 추가 ## 📢 리뷰어 참고 사항 (To Reviewers) - 기존 토큰에는 `adminId` 클레임이 없으므로 반드시 재로그인 필요 - `sub` 클레임은 JWT 표준으로 유지, `adminId`는 비즈니스 로직용으로 별도 추가 ## ✅ 체크리스트 (Self Checklist) - [x] 빌드(Build)가 성공적으로 수행되었는가? - [x] 모든 단위 테스트(Unit Test)를 통과하였는가? - [x] 불필요한 로그나 주석을 제거하였는가? - [x] 컨벤션(Clean Architecture, Naming)을 준수하였는가? - [x] 기밀 정보(비밀번호, 키 등)가 하드코딩 되어있지 않은가? ## 📸 스크린샷 / 테스트 로그 (Screenshots/Logs) 빌드 성공: 경고 0개, 오류 0개
seonkyu.kim added 1 commit 2026-02-28 09:35:44 +00:00 
- GenerateAccessToken에서 adminId 클레임을 별도로 추가
- 컨트롤러에서 User.FindFirst("adminId")로 조회 가능하도록 수정

Closes #260
seonkyu.kim added the
Priority
High
Status
In Progress
Type
Bug
 labels 2026-02-28 09:35:52 +00:00
seonkyu.kim self-assigned this 2026-02-28 09:35:55 +00:00
seonkyu.kim added this to the Phase 0: 공통 기반 + 인증 milestone 2026-02-28 09:36:04 +00:00
seonkyu.kim requested review from Owners 2026-02-28 09:36:10 +00:00
seonkyu.kim merged commit b02910a213 into develop 2026-02-28 09:37:18 +00:00
seonkyu.kim deleted branch fix/#260-jwt-adminid-claim 2026-02-28 09:37:42 +00:00
seonkyu.kim added
Status
Done
 and removed
Status
In Progress
 labels 2026-02-28 09:40:20 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
SPMS/Owners
Labels
Clear labels
  
Priority
High
우선순위: 높음

  
Priority
Low
우선순위: 낮음

  
Priority
Medium
우선순위: 보통 [기본]

  
Priority
Non
우선순위: 없음 [처리 시점 유연]

  
Priority
Urgent
우선순위: 긴급 [지연 금지, 즉시 처리]

  
Status
Available
시작 전

  
Status
Done
종료

  
Status
In Progress
진행 중

  
Type
Bug
버그 수정

  
Type
Chore
유지보수

  
Type
Design
디자인

  
Type
Documentation
문서화

  
Type
Feature
새 기능 추가

  
Type
Improvement
기능 개선

  
Type
Refactor
리팩토링

  
Type
Research
연구, 조사

  
Type
Task
할 일

No Label
Priority
High
Priority
Low
Priority
Medium
Priority
Non
Priority
Urgent
Status
Available
Status
Done
Status
In Progress
Type
Bug
Type
Chore
Type
Design
Type
Documentation
Type
Feature
Type
Improvement
Type
Refactor
Type
Research
Type
Task
Milestone
Clear milestone
No items
No Milestone
Phase 0: 공통 기반 + 인증
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
seonkyu.kim
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: SPMS/SPMS_API#261
No description provided.
Delete Branch "fix/#260-jwt-adminid-claim"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?