fix(admin): revive prod admin dashboard (404 unblock)#120
Open
imhyunho99 wants to merge 1 commit into
Open
Conversation
… /admin/dashboard)
prod에서 /onz/admin/login 이 Whitelabel 404로 떨어지던 원인:
SecurityConfig 는 /admin/** 체인을 정의하고 loginPage("/admin/login") 을 가리키지만,
dev 브랜치에 해당 라우트를 처리하는 컨트롤러가 없어 Spring 이 매핑을 찾지 못함.
PR MobileOnz#116 에서 해당 부분만 분리해 가져옴.
- AdminViewController: /admin/login, /admin/dashboard, /admin/dashboard/data 라우터
- AdminDashboardService: 메트릭 집계 (EntityManager 만 의존, 외부 도메인 의존 없음)
- templates/admin/dashboard.html: 차트/테이블 포함 풀 버전
PR MobileOnz#116 의 inquiry/guide/image-pipeline/CI-CD 변경은 의도적으로 제외 — admin 만 살리는 minimum patch.
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
http://onz-cocktail.kr/onz/admin/login이 Whitelabel 404 로 깨져 있는 상태를 unblock 함SecurityConfig가/admin/**체인 +loginPage("/admin/login")을 정의하지만, 해당 라우트를 처리하는 컨트롤러가 dev 에 없음. 그래서 SecurityFilter 통과 후 라우트 매핑에서 404Changed
AdminViewController.java(new)/admin/login,/admin/dashboard,/admin/dashboard/data라우터AdminDashboardService.java(new)EntityManager만 의존, 외부 도메인 의존 없음templates/admin/dashboard.htmldev 의
SecurityConfig,templates/admin/login.html, static admin css/js 는 그대로 사용.Why minimum patch
PR #116 은 admin + inquiry + guide + image-pipeline + CI-CD 가 한 묶음이라 4289 라인 변경. 풀 머지 전까지 prod admin 만 부활시키는 게 목표.
Test plan
http://onz-cocktail.kr/onz/admin/login→ admin 로그인 페이지 렌더 (404 아님)/onz/admin/dashboard차트 + 메트릭 표시🤖 Generated with Claude Code