Merged
Conversation
- middleware에서 관리자 권한 체크 제거 (인증만 체크), Edge Runtime 호환성 확보 - admin layout에서 접근 거부 다이얼로그가 정상 표시되도록 복원 - /api/admin/check 응답에서 userId/discordId 불필요 노출 제거 - admin layout JSON 파싱 에러 방어 코드 추가 - 로그아웃 실패 시 toast 피드백 추가 - admin check + auth/me 병렬 fetch로 성능 개선 Co-Authored-By: Claude <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
- withSentryConfig에 webpack.autoInstrumentMiddleware: false 추가 - Next.js 16 + Sentry v10 환경에서 middleware.js.nft.json 누락 에러 해결 Co-Authored-By: Claude <noreply@anthropic.com>
bbbang105
force-pushed
the
fix/admin-auth-dialog-and-middleware
branch
from
0f680c6 to
9f32d5b
Compare
Next.js 16 + Sentry withSentryConfig 조합에서 middleware.ts가 있으면 middleware.js.nft.json 생성 실패로 Vercel 배포 불가. (Sentry가 자체 middleware proxy를 생성하는데, 사용자 middleware와 충돌) - middleware.ts 삭제 (PR #5에서 Next.js 16 업그레이드 시 삭제한 것이 맞았음) - (user)/layout.tsx: 미인증 시 /login 리다이렉트 추가 - (admin)/layout.tsx: 401 미인증 시 /login 리다이렉트, 비관리자 시 접근 거부 다이얼로그 - next.config.ts 원복 (Sentry 설정 정상 유지) Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Changes
middleware.tsisAdminDiscordIdimport 삭제(admin)/layout.tsx/api/admin/check/route.tsuserId/discordId제거 (정보 노출 방지)CLAUDE.mdDesign Decisions
process.cwd), admin layout의 다이얼로그 UX 보존Test Plan
/admin접근 → "관리자만 접근할 수 있습니다" 다이얼로그 표시/admin접근 → 정상 렌더링/admin접근 →/login리다이렉트🤖 Generated with Claude Code