Auto-deploy from GitHub Actions - 2025-12-14 03:20:12

app/routes.py

@@ -2,6 +2,7 @@ from flask import Blueprint, render_template, request, jsonify, send_from_direct
 from flask import current_app
 from flask_login import login_user, logout_user, login_required, current_user
 from werkzeug.utils import secure_filename
+from werkzeug.routing import BuildError
 from app.database import db, UploadedFile, User, ChatSession, ChatMessage, DocumentChunk, ParentChunk, SystemConfig, EpisodeAnalysis, GraphEntity, GraphRelationship, GraphEvent, ChatbotPrompt
 from app.vector_db import get_vector_db
 from app.gemini_client import get_gemini_client
@@ -179,12 +180,21 @@ def save_admin_menu_config(config_obj):
 @main_bp.app_context_processor
 def inject_admin_menu():
     """모든 템플릿에서 admin_menu 사용 가능하도록 주입"""
+    def safe_url_for(endpoint, **values):
+        """템플릿에서 endpoint 미존재로 500이 나지 않도록 안전한 url_for 제공"""
+        try:
+            return url_for(endpoint, **values)
+        except BuildError:
+            return '#'
+        except Exception:
+            return '#'
+
     try:
         if getattr(current_user, "is_authenticated", False) and getattr(current_user, "is_admin", False):
-            return {"admin_menu": get_admin_menu_config()}
+            return {"admin_menu": get_admin_menu_config(), "safe_url_for": safe_url_for}
     except Exception:
         pass
-    return {}
+    return {"safe_url_for": safe_url_for}
 
 
 def ensure_chatbot_prompt_table_exists():

templates/_admin_nav.html

@@ -191,6 +191,14 @@
   </style>
 
   {% set _menu = admin_menu if admin_menu is defined else {'sections': [], 'actions': [{'label': '메인으로', 'endpoint': 'main.index'}, {'label': '로그아웃', 'endpoint': 'main.logout'}]} %}
+  {# safe_url_for가 서버에서 주입되지 않은 경우에도 500이 나지 않도록 폴백 #}
+  {% macro nav_url(endpoint) -%}
+    {%- if safe_url_for is defined -%}
+      {{ safe_url_for(endpoint) }}
+    {%- else -%}
+      {{ url_for(endpoint) }}
+    {%- endif -%}
+  {%- endmacro %}
 
   <div class="header">
     <div class="header-title">
@@ -201,20 +209,20 @@
     <div class="header-actions">
       <span style="margin-right: 12px; color: #5f6368;">{{ current_user.nickname or current_user.username }}</span>
 
-      {% for section in _menu.sections %}
+  {% for section in _menu['sections'] %}
         <div class="dropdown">
-          <button type="button" class="dropdown-toggle">{{ section.label }}</button>
+          <button type="button" class="dropdown-toggle">{{ section['label'] }}</button>
           <div class="dropdown-menu">
-            {% for item in section.items %}
-              <a href="{{ url_for(item.endpoint) }}" class="dropdown-item">{{ item.label }}</a>
+            {% for item in section['items'] %}
+              <a href="{{ nav_url(item['endpoint']) }}" class="dropdown-item">{{ item['label'] }}</a>
             {% endfor %}
           </div>
         </div>
       {% endfor %}
 
-      {% for action in _menu.actions %}
-        <a href="{{ url_for(action.endpoint) }}" class="btn" style="padding: 8px 16px; font-size: 14px; {% if not loop.first %}margin-left: 4px;{% endif %}">
-          {{ action.label }}
+      {% for action in _menu['actions'] %}
+        <a href="{{ nav_url(action['endpoint']) }}" class="btn" style="padding: 8px 16px; font-size: 14px; {% if not loop.first %}margin-left: 4px;{% endif %}">
+          {{ action['label'] }}
         </a>
       {% endfor %}
     </div>
@@ -229,15 +237,15 @@
       </div>
       <div class="mobile-menu-user">{{ current_user.nickname or current_user.username }}</div>
       <div class="mobile-menu-items">
-        {% for section in _menu.sections %}
-          <div class="mobile-menu-section">{{ section.label }}</div>
-          {% for item in section.items %}
-            <a href="{{ url_for(item.endpoint) }}" class="mobile-menu-item" onclick="adminNavCloseMobileMenu()">{{ item.label }}</a>
+        {% for section in _menu['sections'] %}
+          <div class="mobile-menu-section">{{ section['label'] }}</div>
+          {% for item in section['items'] %}
+            <a href="{{ nav_url(item['endpoint']) }}" class="mobile-menu-item" onclick="adminNavCloseMobileMenu()">{{ item['label'] }}</a>
           {% endfor %}
         {% endfor %}
         <div class="mobile-menu-section">기타</div>
-        {% for action in _menu.actions %}
-          <a href="{{ url_for(action.endpoint) }}" class="mobile-menu-item" onclick="adminNavCloseMobileMenu()">{{ action.label }}</a>
+        {% for action in _menu['actions'] %}
+          <a href="{{ nav_url(action['endpoint']) }}" class="mobile-menu-item" onclick="adminNavCloseMobileMenu()">{{ action['label'] }}</a>
         {% endfor %}
       </div>
     </div>

templates/admin_menu.html

@@ -183,6 +183,12 @@
         const textarea = document.getElementById('menuJson');
         const alertEl = document.getElementById('alert');
 
+        function escapeHtml(text) {
+            const div = document.createElement('div');
+            div.textContent = text == null ? '' : String(text);
+            return div.innerHTML;
+        }
+
         function showAlert(msg, type) {
             alertEl.className = `alert ${type}`;
             alertEl.textContent = msg;
@@ -192,10 +198,23 @@
             return JSON.stringify(obj, null, 2);
         }
 
+        async function fetchJson(url, options = {}) {
+            const res = await fetch(url, options);
+            const text = await res.text();
+            let data = null;
+            try {
+                data = text ? JSON.parse(text) : null;
+            } catch (e) {
+                // HTML(<!doctype ...) 등이 오면 여기로 들어옴
+                const preview = (text || '').slice(0, 120).replace(/\s+/g, ' ');
+                throw new Error(`Unexpected token '<' 등 JSON 파싱 실패 (status ${res.status}). 응답 미리보기: ${preview}`);
+            }
+            return { res, data };
+        }
+
         async function loadMenu() {
             try {
-                const res = await fetch('/api/admin/menu', { credentials: 'include' });
-                const data = await res.json();
+                const { res, data } = await fetchJson('/api/admin/menu', { credentials: 'include' });
                 if (!res.ok) {
                     showAlert(data.error || `불러오기 실패 (${res.status})`, 'error');
                     return;
@@ -216,13 +235,12 @@
                 return;
             }
             try {
-                const res = await fetch('/api/admin/menu', {
+                const { res, data } = await fetchJson('/api/admin/menu', {
                     method: 'PUT',
                     credentials: 'include',
                     headers: { 'Content-Type': 'application/json' },
                     body: JSON.stringify(obj)
                 });
-                const data = await res.json();
                 if (!res.ok) {
                     showAlert(data.error || `저장 실패 (${res.status})`, 'error');
                     return;
@@ -272,8 +290,7 @@
 
         async function loadEndpoints() {
             try {
-                const res = await fetch('/api/admin/menu/endpoints', { credentials: 'include' });
-                const data = await res.json();
+                const { res, data } = await fetchJson('/api/admin/menu/endpoints', { credentials: 'include' });
                 if (!res.ok) {
                     showAlert(data.error || `endpoint 목록 로드 실패 (${res.status})`, 'error');
                     return;
@@ -281,7 +298,7 @@
                 allEndpoints = data.endpoints || [];
                 renderEndpointList('');
             } catch (e) {
-                showAlert(`endpoint 목록 로드 오류: ${e.message}`, 'error');
+                showAlert(`endpoint 목록 로드 오류: ${e.message}\n(대부분 서버가 최신 코드로 재시작되지 않아 404 HTML이 내려오는 경우입니다)`, 'error');
             }
         }