Privacy-first: remove local storage of inputs from cloud, docker and network tools

Author: dmtkfs

src/tools/cloud-misconfig/CloudMisconfigScanner.jsx

@@ -1,6 +1,5 @@
-import React, { useRef, useState, useEffect } from 'react'
+import React, { useRef, useState } from 'react'
 import AboutSection from '../../components/AboutSection.jsx'
-import { getTemp, setTemp } from '../../utils/storage.js'
 import { downloadTextFile, findingsToMarkdown } from '../../utils/exportUtils.js'
 import {
   analyzeMultiCloudConfig,
@@ -59,35 +58,10 @@ function safeParseJsonWithLocation(text) {
 }
 
 function CloudMisconfigScanner({ onBack }) {
-  const [configText, setConfigText] = useState(() =>
-    getTemp('sw_cloud_config_text', ''),
-  )
-  const [findings, setFindings] = useState(() =>
-    getTemp('sw_cloud_findings', []),
-  )
-  const [hasAnalyzed, setHasAnalyzed] = useState(() =>
-    getTemp('sw_cloud_hasAnalyzed', false),
-  )
-  const [detectedPlatform, setDetectedPlatform] = useState(() =>
-    getTemp('sw_cloud_detectedPlatform', null),
-  )
-
-  // Sync to temp storage (TTL-limited)
-  useEffect(() => {
-    setTemp('sw_cloud_config_text', configText)
-  }, [configText])
-
-  useEffect(() => {
-    setTemp('sw_cloud_findings', findings)
-  }, [findings])
-
-  useEffect(() => {
-    setTemp('sw_cloud_hasAnalyzed', hasAnalyzed)
-  }, [hasAnalyzed])
-
-  useEffect(() => {
-    setTemp('sw_cloud_detectedPlatform', detectedPlatform)
-  }, [detectedPlatform])
+  const [configText, setConfigText] = useState('')
+  const [findings, setFindings] = useState([])
+  const [hasAnalyzed, setHasAnalyzed] = useState(false)
+  const [detectedPlatform, setDetectedPlatform] = useState(null)
 
   // Ephemeral UI state
   const [error, setError] = useState(null)

src/tools/docker-analyzer/DockerAnalyzer.jsx

@@ -1,33 +1,13 @@
-import React, { useRef, useState, useEffect } from 'react'
+import React, { useRef, useState } from 'react'
 import { analyzeDockerfile } from './dockerRules.js'
 import AboutSection from '../../components/AboutSection.jsx'
-import { getTemp, setTemp } from '../../utils/storage.js'
 import { downloadTextFile, findingsToMarkdown } from '../../utils/exportUtils.js'
 
 function DockerAnalyzer({ onBack }) {
-  // Persisted state (temp storage with TTL)
-  const [dockerfileText, setDockerfileText] = useState(() =>
-    getTemp('sw_docker_text', ''),
-  )
-  const [findings, setFindings] = useState(() =>
-    getTemp('sw_docker_findings', []),
-  )
-  const [hasAnalyzed, setHasAnalyzed] = useState(() =>
-    getTemp('sw_docker_hasAnalyzed', false),
-  )
-
-  // Sync to temp storage
-  useEffect(() => {
-    setTemp('sw_docker_text', dockerfileText)
-  }, [dockerfileText])
-
-  useEffect(() => {
-    setTemp('sw_docker_findings', findings)
-  }, [findings])
-
-  useEffect(() => {
-    setTemp('sw_docker_hasAnalyzed', hasAnalyzed)
-  }, [hasAnalyzed])
+  // Ephemeral state (in-memory only)
+  const [dockerfileText, setDockerfileText] = useState('')
+  const [findings, setFindings] = useState([])
+  const [hasAnalyzed, setHasAnalyzed] = useState(false)
 
   // Ephemeral UI state
   const [error, setError] = useState(null)

src/tools/network-exposure-map/NetworkExposureMap.jsx

@@ -1,4 +1,4 @@
-import React, { useState, useMemo, useRef, useEffect } from 'react'
+import React, { useState, useMemo, useRef } from 'react'
 import {
   parseScanFile,
   SAMPLE_SCAN_XML,
@@ -10,7 +10,6 @@ import {
 } from './scanParsers.js'
 import AboutSection from '../../components/AboutSection.jsx'
 import { downloadTextFile } from '../../utils/exportUtils.js'
-import {getTemp, setTemp, getPersistent, setPersistent } from '../../utils/storage.js'
 
 function riskBadgeClasses(level) {
   switch (level) {
@@ -168,71 +167,27 @@ function hostsToCsvReport(hosts) {
 }
 
 function NetworkExposureMap({ onBack }) {
-  const [hosts, setHosts] = useState(() => 
-    getTemp('sw_netmap_hosts', []),
-  )
+  const [hosts, setHosts] = useState([])
   const [error, setError] = useState(null)
   const [lastSource, setLastSource] = useState(null)
-  const [manualInput, setManualInput] = useState(() =>
-    getTemp('sw_netmap_manual', ''),
-  )
-  const [showOnlyElevated, setShowOnlyElevated] = useState(() =>
-    getPersistent('sw_netmap_showElevated', false),
-  )
+  const [manualInput, setManualInput] = useState('')
+  const [showOnlyElevated, setShowOnlyElevated] = useState(false)
   const [exportMessage, setExportMessage] = useState(null)
   const [searchQuery, setSearchQuery] = useState('')
-  const [onlyUpHosts, setOnlyUpHosts] = useState(() =>
-    getPersistent('sw_netmap_onlyUp', false),
-  )
-  const [viewMode, setViewMode] = useState(() =>
-    getPersistent('sw_netmap_viewMode', 'hosts'),
-  )
+  const [onlyUpHosts, setOnlyUpHosts] = useState(false)
+  const [viewMode, setViewMode] = useState('hosts')
 
   // Host detail side panel
   const [selectedHost, setSelectedHost] = useState(null)
 
   // Custom high-risk markers (UI-only, persisted locally)
-  const [userHighRiskPorts, setUserHighRiskPorts] = useState(() =>
-    getPersistent('sw_netmap_custom_ports', []),
-  )
-  const [userHighRiskServices, setUserHighRiskServices] = useState(() =>
-    getPersistent('sw_netmap_custom_services', []),
-  )
+  const [userHighRiskPorts, setUserHighRiskPorts] = useState([])
+  const [userHighRiskServices, setUserHighRiskServices] = useState([])
   const [newHighPort, setNewHighPort] = useState('')
   const [newHighService, setNewHighService] = useState('')
 
   const fileInputRef = useRef(null)
 
-  // Sync temp state to storage
-  useEffect(() => {
-    setTemp('sw_netmap_hosts', hosts)
-  }, [hosts])
-
-  useEffect(() => {
-    setTemp('sw_netmap_manual', manualInput)
-  }, [manualInput])
-
-  // Sync persistent prefs to storage
-  useEffect(() => {
-    setPersistent('sw_netmap_showElevated', showOnlyElevated)
-  }, [showOnlyElevated])
-
-  useEffect(() => {
-    setPersistent('sw_netmap_onlyUp', onlyUpHosts)
-  }, [onlyUpHosts])
-
-  useEffect(() => {
-    setPersistent('sw_netmap_viewMode', viewMode)
-  }, [viewMode])
-
-  useEffect(() => {
-    setPersistent('sw_netmap_custom_ports', userHighRiskPorts)
-  }, [userHighRiskPorts])
-
-  useEffect(() => {
-    setPersistent('sw_netmap_custom_services', userHighRiskServices)
-  }, [userHighRiskServices])
-
   // Helper: custom high-risk logic (adds on top of built-in)
   const isCustomHighRisk = (port) => {
     if (!port) return false
@@ -554,20 +509,23 @@ function NetworkExposureMap({ onBack }) {
                 )
               </p>
 
-              <label className="block">
-                <span className="sr-only">Upload scan file</span>
+              <label className="inline-flex items-center">
+                <span className="inline-flex items-center justify-center
+                                px-3 py-1.5 rounded-full
+                                text-xs font-semibold
+                                bg-emerald-500/90 text-slate-950
+                                hover:bg-emerald-400/90 cursor-pointer">
+                  Browse…
+                </span>
+
                 <input
                   ref={fileInputRef}
                   type="file"
                   accept=".xml,.json,.txt"
                   onChange={handleFileChange}
-                  className="block w-full text-xs text-slate-200
-                             file:mr-3 file:py-1.5 file:px-3 file:rounded-full
-                             file:border-0 file:text-xs file:font-semibold
-                             file:bg-emerald-500/90 file:text-slate-950
-                             hover:file:bg-emerald-400/90 cursor-pointer"
+                  className="sr-only"
                 />
-              </label>
+            </label>
 
               {lastSource && (
                 <p className="mt-2 text-[0.7rem] text-slate-500">