diff --git a/plans/250831_todo_and_tech_debt.md b/plans/250831_todo_and_tech_debt.md index e0f290a..56bbde8 100644 --- a/plans/250831_todo_and_tech_debt.md +++ b/plans/250831_todo_and_tech_debt.md @@ -1,11 +1,11 @@ -# TODO 및 기술 부채 정리 +# 기술 부채 정리 ## 작성일: 2025년 8월 31일 ## 최종 수정: 2025년 9월 4일 ## 250831 Slack OAuth 구현 완료 후 남은 과제 -### 1. 임시 코드 방식 통일 필요 🔴 +### 1. 임시 코드 방식 통일 🔴 **현재 상황**: - Gmail 로그인: 임시 코드 → `/auth/verify` → JWT @@ -19,7 +19,7 @@ **개선 방안**: ```javascript -// Option 1: JWT 직접 전달 (보안 검토 필요) +// Option 1: JWT 직접 전달 (보안 검토) return RedirectResponse(f"{redirect_uri}#token={jwt_token}") // Option 2: Secure cookie 사용 @@ -35,7 +35,7 @@ response.set_cookie("auth_token", jwt_token, secure=True, httponly=True) - Docker 이미지 재빌드 완료 - 의존성 확인 완료 -## 2. 긴급 해결 필요 사항 🔴 +## 2. 긴급 해결 사항 🔴 ### ✅ Git Merge Conflict 해결 완료 **해결일**: 2025-08-31 16:15 @@ -67,7 +67,7 @@ SlackWorkspace.workspace_id → workspaces.id (실제로는 company_id) 3. FK 관계 재설정 4. 모델 파일 통일 -### 3. 환경 변수 하드코딩 제거 필요 🟡 +### 3. 환경 변수 하드코딩 제거 🟡 **현재 하드코딩된 값들**: ```python @@ -80,10 +80,10 @@ fetch('https://auth.ro-being.com/auth/verify', { # Line 79 ``` **개선 방안**: -- 환경변수로 추출 필요 +- 환경변수로 추출 - 개발/스테이징/프로덕션 환경 분리 -### 4. 사용자 매핑 로직 개선 필요 🟡 +### 4. 사용자 매핑 로직 개선 🟡 **현재 로직**: 1. slack_user_mapping 확인