fix: harden chatbot and public proxy fallbacks

Author: Nguyen Son

README.md

@@ -126,6 +126,16 @@ cd frontend && BASE_URL=http://localhost:3001 npm run test:e2e:portfolio
 - `GROK_ENABLED=false` by default
 - Set `GROK_ENABLED=true` together with `GROK_API_KEY` to enable Grok in local/dev or Docker
 - Frontend chatbot health is driven by `/api/chatbot/health`
+- Public health responses are sanitized; raw provider errors and user email addresses are not returned to the browser.
+
+## CI/CD registry secrets
+
+The GitHub Actions pipeline publishes to GHCR automatically. Docker Hub publish is enabled when these repository secrets exist:
+
+- `DOCKERHUB_USERNAME`
+- `DOCKERHUB_TOKEN`
+
+If `DOCKERHUB_NAMESPACE` is not set as a repository variable, the workflow uses `DOCKERHUB_USERNAME` as the Docker Hub namespace.
 
 ## Payments
 

README_VN.md

@@ -9,11 +9,11 @@
 
 Dự án được xây dựng trên mô hình Micro-monolith hiện đại:
 
-- **Backend**: Spring Boot 3.3, Java 17, Spring Security (JWT), Spring Data JPA.
-- **Frontend**: Next.js 14 (App Router), TypeScript, Tailwind CSS, Shadcn/UI, React Query.
-- **Mobile**: React Native (Expo), Lucide Icons, Zustand.
+- **Backend**: Spring Boot 3.2, Java 17, Spring Security (JWT), Spring Data JPA.
+- **Frontend**: Next.js 16 (App Router), React 19, TypeScript, Tailwind CSS, React Query.
+- **Mobile**: Expo SDK 55, React Native 0.83, Lucide Icons, Zustand.
 - **Database & DevOps**: MySQL 8, Docker, Docker Compose, GitHub Actions (CI/CD).
-- **Security**: Bucket4j (Rate Limiting), JWT Stateless Auth, CSP Headers.
+- **Security**: Rate limiting, JWT stateless auth, CSP/HSTS/Permissions-Policy headers.
 
 ## Tính năng Nổi bật
 
@@ -51,13 +51,15 @@ docker compose up -d
 ```
 
 Hệ thống sẽ khả dụng tại:
-- Frontend: `http://localhost:3000`
+- Frontend: `http://localhost:3001`
 - Backend API: `http://localhost:8080/api`
-- Swagger UI: `http://localhost:8080/swagger-ui.html`
+- Swagger UI: `http://localhost:8080/api/swagger-ui.html`
 
 ## Bảo mật & Độ tin cậy
 - **Rate Limiting**: Chống tấn công brute-force ở các điểm nhạy cảm (Login, Register, Chatbot).
 - **CI/CD**: Tự động chạy Unit Test, Integration Test (MySQL), E2E Test (Playwright) và Security Scan (Trivy) trên mỗi lần push.
+- **Chatbot an toàn**: Health endpoint công khai không trả lỗi provider thô; context gửi sang Grok không kèm email người dùng.
+- **Docker Hub**: Pipeline publish thật khi repository secrets `DOCKERHUB_USERNAME` và `DOCKERHUB_TOKEN` đã được cấu hình.
 
 ## Tài liệu chi tiết
 

backend/src/main/java/com/bookstore/config/RateLimitingFilter.java

@@ -60,6 +60,12 @@ public void doFilter(ServletRequest servletRequest, ServletResponse servletRespo
 
         String clientIp = getClientIP(request);
         String requestUri = request.getRequestURI();
+
+        if (isHealthEndpoint(requestUri)) {
+            filterChain.doFilter(servletRequest, servletResponse);
+            return;
+        }
+
         String key = clientIp + ":" + requestUri;
 
         RateLimitConfig config = getRateLimitConfig(request.getMethod(), requestUri);
@@ -124,6 +130,19 @@ private boolean isPublicCatalogEndpoint(String uri) {
                 || uri.equals("/api/coupons/available")
                 || uri.equals("/coupons/available");
     }
+
+    private boolean isHealthEndpoint(String uri) {
+        return uri.equals("/api/health")
+                || uri.startsWith("/api/health/")
+                || uri.equals("/health")
+                || uri.startsWith("/health/")
+                || uri.equals("/api/actuator/health")
+                || uri.startsWith("/api/actuator/health/")
+                || uri.equals("/actuator/health")
+                || uri.startsWith("/actuator/health/")
+                || uri.equals("/api/chatbot/health")
+                || uri.equals("/chatbot/health");
+    }
 
     private String getClientIP(HttpServletRequest request) {
         String xForwardedFor = request.getHeader("X-Forwarded-For");

backend/src/main/java/com/bookstore/controller/ChatbotController.java

@@ -14,6 +14,7 @@
 import lombok.RequiredArgsConstructor;
 import org.springframework.http.ResponseEntity;
 import org.springframework.security.core.annotation.AuthenticationPrincipal;
+import org.springframework.security.access.prepost.PreAuthorize;
 import org.springframework.web.bind.annotation.DeleteMapping;
 import org.springframework.web.bind.annotation.GetMapping;
 import org.springframework.web.bind.annotation.PathVariable;
@@ -78,6 +79,7 @@ public ResponseEntity<ApiResponse<Void>> submitFeedback(
 
     @GetMapping("/stats")
     @Operation(summary = "Lay thong ke chatbot (Admin)")
+    @PreAuthorize("hasRole('ADMIN')")
     public ResponseEntity<ApiResponse<Map<String, Object>>> getChatbotStats() {
         return ResponseEntity.ok(ApiResponse.success(chatbotService.getChatbotStats()));
     }

backend/src/main/java/com/bookstore/service/AbstractChatbotService.java

@@ -49,7 +49,7 @@ public abstract class AbstractChatbotService implements ChatbotService {
         - Thân thiện, gần gũi như đang trò chuyện với người bạn
         - Sử dụng tiếng Việt, có thể xen lẫn tiếng Anh cho tên sách/tác giả
         - Trả lời NGẮN GỌN (dưới 200 từ) trừ khi cần chi tiết
-        - Nếu không biết thì gợi ý hotline: 1900-xxxx hoặc email: support@bookstore.com
+        - Nếu không biết thì gợi ý trang Liên hệ hoặc email support@bookstore.com
 
         CÁC TÍNH NĂNG CỦA CỬA HÀNG:
         - Giao hàng toàn quốc, miễn phí vận chuyển cho đơn từ 200.000đ
@@ -167,10 +167,10 @@ protected String getFallbackResponse() {
 
             Bạn có thể:
             - Thử hỏi lại câu hỏi của mình
-            - Liên hệ hotline: 1900-xxxx để được hỗ trợ nhanh hơn
-            - Email: support@bookstore.com
+            - Mở trang Liên hệ để gửi yêu cầu hỗ trợ
+            - Email support@bookstore.com nếu cần kiểm tra đơn hàng gấp
 
-            Cảm ơn bạn đã thông cảm!
+            Cảm ơn bạn đã thông cảm.
             """;
     }
 
@@ -180,8 +180,8 @@ protected String getDisabledResponse() {
 
             Bạn vẫn có thể:
             - Duyệt danh mục sách và tìm kiếm trực tiếp trên cửa hàng
-            - Liên hệ hotline: 1900-xxxx
-            - Email: support@bookstore.com
+            - Mở trang Liên hệ để gửi yêu cầu hỗ trợ
+            - Email support@bookstore.com nếu cần kiểm tra đơn hàng gấp
 
             Khi trợ lý được bật lại, mình sẽ hỗ trợ tư vấn chi tiết hơn.
             """;
@@ -235,8 +235,9 @@ protected List<Map<String, String>> buildMessages(List<ChatMessage> history, Str
 
     protected String buildUserContext(User user) {
         StringBuilder context = new StringBuilder("\n\nTHÔNG TIN KHÁCH HÀNG:");
-        context.append("\n- Tên: ").append(user.getFullName());
-        context.append("\n- Email: ").append(user.getEmail());
+        String displayName = user.getFullName();
+        context.append("\n- Tên hiển thị: ")
+                .append(displayName == null || displayName.isBlank() ? "Khách hàng đã đăng nhập" : displayName);
 
         try {
             var orders = orderRepository.findByUserId(user.getId(), PageRequest.of(0, 1));

backend/src/main/java/com/bookstore/service/GrokChatbotService.java

@@ -142,12 +142,11 @@ protected int getMaxContextMessages() {
     @Override
     protected String getStatusMessage() {
         if (grokApiKey == null || grokApiKey.isBlank()) {
-            return "Grok đang bật nhưng chưa có API key trong environment.";
+            return "Trợ lý AI đang ở chế độ dự phòng do cấu hình provider chưa hoàn tất.";
         }
 
         if (lastFailureAt != null && (lastSuccessAt == null || lastFailureAt.isAfter(lastSuccessAt))) {
-            return "Grok đã được cấu hình nhưng lần gọi gần nhất thất bại: "
-                    + (lastFailureMessage != null ? lastFailureMessage : "không rõ nguyên nhân");
+            return "Trợ lý AI đã được cấu hình nhưng lần gọi gần nhất chưa ổn định. Chatbot đang dùng chế độ dự phòng.";
         }
 
         return "Grok đã được cấu hình và sẵn sàng trả lời.";

backend/src/test/java/com/bookstore/controller/ChatbotControllerTest.java

@@ -141,6 +141,14 @@ void getChatbotStats_asAdmin() throws Exception {
                 .andExpect(jsonPath("$.data.totalConversations").value(100));
     }
 
+    @Test
+    @WithMockUser(username = "customer@example.com", roles = {"CUSTOMER"})
+    @DisplayName("GET /api/chatbot/stats - rejects non-admin users")
+    void getChatbotStats_asCustomer_forbidden() throws Exception {
+        mockMvc.perform(get("/chatbot/stats"))
+                .andExpect(status().isForbidden());
+    }
+
     @Test
     @DisplayName("POST /api/chatbot/message - returns 401 when not authenticated")
     void sendMessage_unauthenticated() throws Exception {

backend/src/test/java/com/bookstore/service/ChatbotServiceHealthTest.java

@@ -66,7 +66,9 @@ void grokServiceReportsDegradedHealthWithoutApiKey() {
                 .containsEntry("service", "Grok AI Chatbot")
                 .containsEntry("model", "grok-3")
                 .containsEntry("providerEnabled", "true");
-        assertThat(health.get("message")).contains("API key");
+        assertThat(health.get("message"))
+                .contains("chế độ dự phòng")
+                .doesNotContain("API key");
     }
 
     @Test
@@ -98,12 +100,15 @@ void grokServiceKeepsStartupHealthyButMarksRecentFailures() {
 
         assertThat(generatedReply.content())
                 .contains("support@bookstore.com")
-                .contains("1900-xxxx");
+                .doesNotContain("1900-xxxx");
         assertThat(generatedReply.providerResponse()).isFalse();
         assertThat(health)
                 .containsEntry("status", "DEGRADED")
                 .containsEntry("providerEnabled", "true");
-        assertThat(health.get("message")).contains("network down");
+        assertThat(health.get("message"))
+                .contains("chế độ dự phòng")
+                .doesNotContain("network down")
+                .doesNotContain("https://api.x.ai");
     }
 
     @Test
@@ -136,6 +141,31 @@ void grokServiceTreatsEmptyProviderContentAsFallback() {
         assertThat(service.getHealthStatus()).containsEntry("status", "DEGRADED");
     }
 
+    @Test
+    @SuppressWarnings("unchecked")
+    void userContextSentToProviderDoesNotIncludeEmail() {
+        GrokChatbotService service = newGrokService();
+        User user = new User();
+        user.setId(17L);
+        user.setEmail("customer@example.com");
+        user.setFirstName("Nguyễn Khách");
+
+        List<Map<String, String>> messages = ReflectionTestUtils.invokeMethod(
+                service,
+                "buildMessages",
+                List.of(),
+                "Tư vấn giúp tôi một cuốn sách kinh doanh",
+                user
+        );
+
+        assertThat(messages).isNotNull();
+        String systemPrompt = messages.get(0).get("content");
+        assertThat(systemPrompt)
+                .contains("Nguyễn Khách")
+                .doesNotContain("customer@example.com")
+                .doesNotContain("Email:");
+    }
+
     private GrokChatbotService newGrokService() {
         return new GrokChatbotService(
                 conversationRepository,

docs/architecture-and-cicd-vn.md

@@ -100,7 +100,7 @@ stateDiagram-v2
    - Publish image lên GHCR và Docker Hub.
    - Tag chuyên nghiệp theo semver:
      - `latest`
-     - `v1.0.0`
+     - `v1.1.2`
      - `v1`
 5. **Render deploy**
    - Gọi deploy hooks sau khi toàn bộ gate chính đã xanh.

docs/architecture-and-cicd.md

@@ -100,7 +100,7 @@ stateDiagram-v2
    - Publishes images to GHCR and Docker Hub.
    - Public tags follow semver-first naming:
      - `latest`
-     - `v1.0.0`
+     - `v1.1.2`
      - `v1`
 5. **Render deploy**
    - Triggers Render deploy hooks after the required gates pass.