jramos/cve-dashboard
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix History tab crash: coerce Ivanti note fields to strings before rendering

Browse Source 
PostgreSQL + Ivanti API enrichment can return non-string values
(objects/arrays) for currentStateUserNotes and similar fields.
React crashes silently (blank page, no console error) when trying
to render non-string values as children. Same root cause pattern
as Bug 3 in ivanti-panel-bugs-2026-05-12.

Added safeText() wrapper that coerces any non-string truthy value
to a JSON string before rendering in the History tab notes section.
This commit is contained in:
Jordan Ramos
2026-05-13 12:01:52 -06:00
parent 671894ff5f
commit 870c0e247a
1 changed files with 9 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
frontend/src/components/pages/ReportingPage.js
View File

@@ -4087,11 +4087,16 @@ function FpEditModal({ open, onClose, submission, queueItems, onSuccess }) {
<div>
{/* Ivanti reviewer notes (rework/approval/previous state feedback) */}
{(() => {
const safeText = (val) => {
if (!val) return null;
if (typeof val === 'string') return val;
try { return JSON.stringify(val); } catch { return String(val); }
};
const notes = [
submission.ivanti_rework_note && { label: 'Rework Note', text: submission.ivanti_rework_note },
submission.ivanti_approval_note && { label: 'Approval Note', text: submission.ivanti_approval_note },
submission.ivanti_current_state_notes && { label: 'Current State Notes', text: submission.ivanti_current_state_notes },
submission.ivanti_previous_state_notes && { label: 'Previous State Notes', text: submission.ivanti_previous_state_notes },
safeText(submission.ivanti_rework_note) && { label: 'Rework Note', text: safeText(submission.ivanti_rework_note) },
safeText(submission.ivanti_approval_note) && { label: 'Approval Note', text: safeText(submission.ivanti_approval_note) },
safeText(submission.ivanti_current_state_notes) && { label: 'Current State Notes', text: safeText(submission.ivanti_current_state_notes) },
safeText(submission.ivanti_previous_state_notes) && { label: 'Previous State Notes', text: safeText(submission.ivanti_previous_state_notes) },
].filter(Boolean);
return notes.length > 0 ? notes.map((note, idx) => (
<div key={idx} style={{