From 00168e009e1de38acac360a0bfe64569b8c03ae8 Mon Sep 17 00:00:00 2001
From: shankar0123 <skreddy040@gmail.com>
Date: Mon, 27 Apr 2026 03:05:52 +0000
Subject: [PATCH] =?UTF-8?q?M-029=20Pass=203=20batch=20B:=20T-1=20tests=20f?=
 =?UTF-8?q?or=204=20detail=20pages=20=E2=80=94=20XSS=20hardening?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Continues Pass 3. Each detail page has its own narrow attack surface

(subject DN, last_test_message, error_message) that the test exercises

with literal <script> payloads in every text field.

Test files added:

  - CertificateDetailPage.test.tsx  cert subject / SANs / serial / PEM

                                     across 7 sidecar queries (getCertificate,

                                     getCertificateVersions, getTargets,

                                     getProfile, getProfiles, getRenewalPolicies,

                                     getJobs all mocked in beforeEach)

  - IssuerDetailPage.test.tsx       issuer name / type / config / last_test_message

                                     (router-aware test using Routes + useParams)

  - TargetDetailPage.test.tsx       target name / config / last_test_message

                                     (router-aware test pattern)

  - JobDetailPage.test.tsx          job error_message / type / details

                                     (3-query mock: getJob + getJobVerification +

                                     getAuditEvents)

Closes 9 of 14 T-1-deferred pages toward M-029 Pass 3 completion (5 batch A,

+ 4 batch B = 9; 5 to go in batch C).
---
 web/src/pages/CertificateDetailPage.test.tsx | 115 +++++++++++++++++++
 web/src/pages/IssuerDetailPage.test.tsx      |  84 ++++++++++++++
 web/src/pages/JobDetailPage.test.tsx         |  88 ++++++++++++++
 web/src/pages/TargetDetailPage.test.tsx      |  85 ++++++++++++++
 4 files changed, 372 insertions(+)
 create mode 100644 web/src/pages/CertificateDetailPage.test.tsx
 create mode 100644 web/src/pages/IssuerDetailPage.test.tsx
 create mode 100644 web/src/pages/JobDetailPage.test.tsx
 create mode 100644 web/src/pages/TargetDetailPage.test.tsx

diff --git a/web/src/pages/CertificateDetailPage.test.tsx b/web/src/pages/CertificateDetailPage.test.tsx
new file mode 100644
index 0000000..da61e5e
--- /dev/null
+++ b/web/src/pages/CertificateDetailPage.test.tsx
@@ -0,0 +1,115 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { render, screen, waitFor, cleanup } from '@testing-library/react';
+import { QueryClient, QueryClientProvider } from '@tanstack/react-query';
+import { MemoryRouter, Route, Routes } from 'react-router-dom';
+import type { ReactNode } from 'react';
+
+// -----------------------------------------------------------------------------
+// M-029 Pass 3 (Audit M-026): CertificateDetailPage XSS-hardening + render
+// coverage.
+//
+// CertificateDetailPage surfaces cert subject DN, SANs, issuer DN, version
+// history, deployment job error messages — every one of these is either
+// CSR-controlled (subject / SANs) or upstream-CA / target-side error text.
+// The M-004 MCP fence handles inside-LLM safety; this test pins GUI XSS
+// safety on the same data.
+// -----------------------------------------------------------------------------
+
+vi.mock('../api/client', () => ({
+  getCertificate: vi.fn(),
+  getCertificateVersions: vi.fn(),
+  getTargets: vi.fn(),
+  getProfile: vi.fn(),
+  getProfiles: vi.fn(),
+  getJobs: vi.fn(),
+  getRenewalPolicies: vi.fn(),
+  triggerRenewal: vi.fn(),
+  triggerDeployment: vi.fn(),
+  archiveCertificate: vi.fn(),
+  revokeCertificate: vi.fn(),
+  updateCertificate: vi.fn(),
+  downloadCertificatePEM: vi.fn(),
+  exportCertificatePKCS12: vi.fn(),
+}));
+
+import CertificateDetailPage from './CertificateDetailPage';
+import * as client from '../api/client';
+
+function renderRoute(ui: ReactNode, path = '/certificates/mc-xss-001') {
+  const qc = new QueryClient({
+    defaultOptions: { queries: { retry: false, gcTime: 0, staleTime: 0 } },
+  });
+  return render(
+    <QueryClientProvider client={qc}>
+      <MemoryRouter initialEntries={[path]}>
+        <Routes>
+          <Route path="/certificates/:id" element={ui} />
+        </Routes>
+      </MemoryRouter>
+    </QueryClientProvider>,
+  );
+}
+
+const xssPayload = '<script data-xss="cert-detail">window.__xss_pwned__=1;</script>';
+
+const xssCert = {
+  id: 'mc-xss-001',
+  name: xssPayload,
+  common_name: xssPayload,
+  sans: [xssPayload, 'plain.example.com'],
+  status: 'Active',
+  environment: xssPayload,
+  issuer_id: 'iss-xss',
+  certificate_profile_id: 'cp-xss',
+  owner_id: 'o-xss',
+  team_id: 't-xss',
+  renewal_policy_id: 'rp-xss',
+  expires_at: new Date(Date.now() + 30 * 86400000).toISOString(),
+  created_at: new Date().toISOString(),
+  not_before: new Date(Date.now() - 86400000).toISOString(),
+  not_after: new Date(Date.now() + 30 * 86400000).toISOString(),
+  serial_number: xssPayload,
+  fingerprint_sha256: xssPayload,
+  pem_encoded: xssPayload,
+};
+
+describe('CertificateDetailPage — render + XSS hardening (M-026 / M-029 Pass 3)', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+    cleanup();
+    delete (window as unknown as { __xss_pwned__?: number }).__xss_pwned__;
+
+    // Wire defaults for the sidecar queries — every page render fans out
+    // 7+ queries and any unmocked call will reject and surface an error
+    // boundary instead of the page body.
+    vi.mocked(client.getCertificate).mockResolvedValue(xssCert as never);
+    vi.mocked(client.getCertificateVersions).mockResolvedValue([] as never);
+    vi.mocked(client.getTargets).mockResolvedValue({ data: [], total: 0, page: 1, per_page: 50 } as never);
+    vi.mocked(client.getProfile).mockResolvedValue({ id: 'cp-xss', name: 'Profile' } as never);
+    vi.mocked(client.getProfiles).mockResolvedValue({ data: [], total: 0, page: 1, per_page: 50 } as never);
+    vi.mocked(client.getRenewalPolicies).mockResolvedValue({ data: [], total: 0, page: 1, per_page: 500 } as never);
+    vi.mocked(client.getJobs).mockResolvedValue({ data: [], total: 0, page: 1, per_page: 50 } as never);
+  });
+
+  it('renders the page when getCertificate resolves', async () => {
+    vi.mocked(client.getCertificate).mockResolvedValue({ ...xssCert, common_name: 'plain.example.com' } as never);
+    renderRoute(<CertificateDetailPage />);
+    await waitFor(() => {
+      expect(screen.getByText('plain.example.com')).toBeInTheDocument();
+    });
+  });
+
+  it('does NOT execute <script> payloads in cert subject / SANs / serial / pem', async () => {
+    renderRoute(<CertificateDetailPage />);
+    await waitFor(() => {
+      expect(document.body.textContent ?? '').toContain('<script data-xss="cert-detail">');
+    });
+
+    const liveScripts = document.querySelectorAll('script[data-xss="cert-detail"]');
+    expect(liveScripts.length, 'cert fields must not inject a live <script>').toBe(0);
+    expect(
+      (window as unknown as { __xss_pwned__?: number }).__xss_pwned__,
+      'cert <script> body must not have executed',
+    ).toBeUndefined();
+  });
+});
diff --git a/web/src/pages/IssuerDetailPage.test.tsx b/web/src/pages/IssuerDetailPage.test.tsx
new file mode 100644
index 0000000..f2c787d
--- /dev/null
+++ b/web/src/pages/IssuerDetailPage.test.tsx
@@ -0,0 +1,84 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { render, screen, waitFor, cleanup } from '@testing-library/react';
+import { QueryClient, QueryClientProvider } from '@tanstack/react-query';
+import { MemoryRouter, Route, Routes } from 'react-router-dom';
+import type { ReactNode } from 'react';
+
+// -----------------------------------------------------------------------------
+// M-029 Pass 3 (Audit M-026): IssuerDetailPage XSS-hardening + render coverage.
+//
+// IssuerDetailPage surfaces the issuer's name, type, config blob keys, and
+// last_test_message (operator-supplied or upstream-CA-supplied error string).
+// A misconfigured-on-purpose CA or MITM that injects a <script> payload into
+// the test-result message must not execute when surfaced in the GUI.
+// -----------------------------------------------------------------------------
+
+vi.mock('../api/client', () => ({
+  getIssuer: vi.fn(),
+  testIssuerConnection: vi.fn(),
+  getCertificates: vi.fn(),
+}));
+
+import IssuerDetailPage from './IssuerDetailPage';
+import * as client from '../api/client';
+
+function renderRoute(ui: ReactNode, path = '/issuers/iss-xss-001') {
+  const qc = new QueryClient({
+    defaultOptions: { queries: { retry: false, gcTime: 0, staleTime: 0 } },
+  });
+  return render(
+    <QueryClientProvider client={qc}>
+      <MemoryRouter initialEntries={[path]}>
+        <Routes>
+          <Route path="/issuers/:id" element={ui} />
+        </Routes>
+      </MemoryRouter>
+    </QueryClientProvider>,
+  );
+}
+
+const xssPayload = '<script data-xss="issuer-detail">window.__xss_pwned__=1;</script>';
+
+const xssIssuer = {
+  id: 'iss-xss-001',
+  name: xssPayload,
+  type: xssPayload,
+  enabled: true,
+  config: { acme_directory_url: xssPayload, eab_kid: xssPayload },
+  last_tested_at: new Date().toISOString(),
+  last_test_status: 'failed',
+  last_test_message: xssPayload,
+};
+
+describe('IssuerDetailPage — render + XSS hardening (M-026 / M-029 Pass 3)', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+    cleanup();
+    delete (window as unknown as { __xss_pwned__?: number }).__xss_pwned__;
+  });
+
+  it('renders the page header when getIssuer resolves', async () => {
+    vi.mocked(client.getIssuer).mockResolvedValue({ ...xssIssuer, name: 'Plain Name', type: 'acme' } as never);
+    vi.mocked(client.getCertificates).mockResolvedValue({ data: [], total: 0, page: 1, per_page: 50 } as never);
+    renderRoute(<IssuerDetailPage />);
+    await waitFor(() => {
+      expect(screen.getByText('Plain Name')).toBeInTheDocument();
+    });
+  });
+
+  it('does NOT execute <script> payloads in name / config / test message', async () => {
+    vi.mocked(client.getIssuer).mockResolvedValue(xssIssuer as never);
+    vi.mocked(client.getCertificates).mockResolvedValue({ data: [], total: 0, page: 1, per_page: 50 } as never);
+    renderRoute(<IssuerDetailPage />);
+    await waitFor(() => {
+      expect(document.body.textContent ?? '').toContain('<script data-xss="issuer-detail">');
+    });
+
+    const liveScripts = document.querySelectorAll('script[data-xss="issuer-detail"]');
+    expect(liveScripts.length, 'issuer fields must not inject a live <script>').toBe(0);
+    expect(
+      (window as unknown as { __xss_pwned__?: number }).__xss_pwned__,
+      'issuer <script> body must not have executed',
+    ).toBeUndefined();
+  });
+});
diff --git a/web/src/pages/JobDetailPage.test.tsx b/web/src/pages/JobDetailPage.test.tsx
new file mode 100644
index 0000000..f37755f
--- /dev/null
+++ b/web/src/pages/JobDetailPage.test.tsx
@@ -0,0 +1,88 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { render, screen, waitFor, cleanup } from '@testing-library/react';
+import { QueryClient, QueryClientProvider } from '@tanstack/react-query';
+import { MemoryRouter, Route, Routes } from 'react-router-dom';
+import type { ReactNode } from 'react';
+
+// -----------------------------------------------------------------------------
+// M-029 Pass 3 (Audit M-026): JobDetailPage XSS-hardening + render coverage.
+//
+// JobDetailPage surfaces the job's status / type / error_message / verification
+// payload / audit-event fan-out. Job error_message is server-rendered upstream
+// CA error text (ACME, DigiCert, etc.) — those CAs return free-form strings
+// that the M-004 MCP fence handles inside LLM context, but a defensive XSS
+// layer is needed in the GUI rendering path too.
+// -----------------------------------------------------------------------------
+
+vi.mock('../api/client', () => ({
+  getJob: vi.fn(),
+  getJobVerification: vi.fn(),
+  getAuditEvents: vi.fn(),
+}));
+
+import JobDetailPage from './JobDetailPage';
+import * as client from '../api/client';
+
+function renderRoute(ui: ReactNode, path = '/jobs/j-xss-001') {
+  const qc = new QueryClient({
+    defaultOptions: { queries: { retry: false, gcTime: 0, staleTime: 0 } },
+  });
+  return render(
+    <QueryClientProvider client={qc}>
+      <MemoryRouter initialEntries={[path]}>
+        <Routes>
+          <Route path="/jobs/:id" element={ui} />
+        </Routes>
+      </MemoryRouter>
+    </QueryClientProvider>,
+  );
+}
+
+const xssPayload = '<script data-xss="job-detail">window.__xss_pwned__=1;</script>';
+
+const xssJob = {
+  id: 'j-xss-001',
+  type: xssPayload,
+  status: 'Failed',
+  certificate_id: 'mc-xss',
+  agent_id: 'a-xss',
+  error_message: xssPayload,
+  details: { reason: xssPayload },
+  created_at: new Date().toISOString(),
+  updated_at: new Date().toISOString(),
+};
+
+describe('JobDetailPage — render + XSS hardening (M-026 / M-029 Pass 3)', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+    cleanup();
+    delete (window as unknown as { __xss_pwned__?: number }).__xss_pwned__;
+  });
+
+  it('renders the page when getJob resolves', async () => {
+    vi.mocked(client.getJob).mockResolvedValue({ ...xssJob, type: 'renewal', error_message: '' } as never);
+    vi.mocked(client.getJobVerification).mockResolvedValue(null as never);
+    vi.mocked(client.getAuditEvents).mockResolvedValue({ data: [], total: 0, page: 1, per_page: 10 } as never);
+    renderRoute(<JobDetailPage />);
+    await waitFor(() => {
+      expect(screen.getByText(/j-xss-001/)).toBeInTheDocument();
+    });
+  });
+
+  it('does NOT execute <script> payloads in job error_message / type / details', async () => {
+    vi.mocked(client.getJob).mockResolvedValue(xssJob as never);
+    vi.mocked(client.getJobVerification).mockResolvedValue(null as never);
+    vi.mocked(client.getAuditEvents).mockResolvedValue({ data: [], total: 0, page: 1, per_page: 10 } as never);
+    renderRoute(<JobDetailPage />);
+    await waitFor(() => {
+      expect(document.body.textContent ?? '').toContain('<script data-xss="job-detail">');
+    });
+
+    const liveScripts = document.querySelectorAll('script[data-xss="job-detail"]');
+    expect(liveScripts.length, 'job fields must not inject a live <script>').toBe(0);
+    expect(
+      (window as unknown as { __xss_pwned__?: number }).__xss_pwned__,
+      'job <script> body must not have executed',
+    ).toBeUndefined();
+  });
+});
diff --git a/web/src/pages/TargetDetailPage.test.tsx b/web/src/pages/TargetDetailPage.test.tsx
new file mode 100644
index 0000000..24643a2
--- /dev/null
+++ b/web/src/pages/TargetDetailPage.test.tsx
@@ -0,0 +1,85 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { render, screen, waitFor, cleanup } from '@testing-library/react';
+import { QueryClient, QueryClientProvider } from '@tanstack/react-query';
+import { MemoryRouter, Route, Routes } from 'react-router-dom';
+import type { ReactNode } from 'react';
+
+// -----------------------------------------------------------------------------
+// M-029 Pass 3 (Audit M-026): TargetDetailPage XSS-hardening + render coverage.
+//
+// TargetDetailPage surfaces the target's name, type, config (operator-supplied
+// host/port/path), and last_test_message (target-side error string from a
+// connection probe). A target that returns an attacker-controlled error
+// payload must not execute as script when rendered.
+// -----------------------------------------------------------------------------
+
+vi.mock('../api/client', () => ({
+  getTarget: vi.fn(),
+  getJobs: vi.fn(),
+  updateTarget: vi.fn(),
+  testTargetConnection: vi.fn(),
+}));
+
+import TargetDetailPage from './TargetDetailPage';
+import * as client from '../api/client';
+
+function renderRoute(ui: ReactNode, path = '/targets/t-xss-001') {
+  const qc = new QueryClient({
+    defaultOptions: { queries: { retry: false, gcTime: 0, staleTime: 0 } },
+  });
+  return render(
+    <QueryClientProvider client={qc}>
+      <MemoryRouter initialEntries={[path]}>
+        <Routes>
+          <Route path="/targets/:id" element={ui} />
+        </Routes>
+      </MemoryRouter>
+    </QueryClientProvider>,
+  );
+}
+
+const xssPayload = '<script data-xss="target-detail">window.__xss_pwned__=1;</script>';
+
+const xssTarget = {
+  id: 't-xss-001',
+  name: xssPayload,
+  type: 'nginx',
+  agent_id: 'a-xss',
+  config: { host: xssPayload, path: xssPayload },
+  last_tested_at: new Date().toISOString(),
+  last_test_status: 'failed',
+  last_test_message: xssPayload,
+};
+
+describe('TargetDetailPage — render + XSS hardening (M-026 / M-029 Pass 3)', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+    cleanup();
+    delete (window as unknown as { __xss_pwned__?: number }).__xss_pwned__;
+  });
+
+  it('renders the page header when getTarget resolves', async () => {
+    vi.mocked(client.getTarget).mockResolvedValue({ ...xssTarget, name: 'Plain Name' } as never);
+    vi.mocked(client.getJobs).mockResolvedValue({ data: [], total: 0, page: 1, per_page: 50 } as never);
+    renderRoute(<TargetDetailPage />);
+    await waitFor(() => {
+      expect(screen.getByText('Plain Name')).toBeInTheDocument();
+    });
+  });
+
+  it('does NOT execute <script> payloads in name / config / test message', async () => {
+    vi.mocked(client.getTarget).mockResolvedValue(xssTarget as never);
+    vi.mocked(client.getJobs).mockResolvedValue({ data: [], total: 0, page: 1, per_page: 50 } as never);
+    renderRoute(<TargetDetailPage />);
+    await waitFor(() => {
+      expect(document.body.textContent ?? '').toContain('<script data-xss="target-detail">');
+    });
+
+    const liveScripts = document.querySelectorAll('script[data-xss="target-detail"]');
+    expect(liveScripts.length, 'target fields must not inject a live <script>').toBe(0);
+    expect(
+      (window as unknown as { __xss_pwned__?: number }).__xss_pwned__,
+      'target <script> body must not have executed',
+    ).toBeUndefined();
+  });
+});