monoadmin/the-other-dude
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(frontend): default ping/traceroute target to 8.8.8.8

Browse Source 
The target input showed "8.8.8.8" as placeholder text but the actual
value was empty. Clicking Ping/Traceroute silently returned because
the empty target guard fired. Users saw the placeholder and assumed
the tool was broken.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Jason Staack
2026-03-09 21:57:10 -05:00
parent a3cc35c4b7
commit 394be01145
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
frontend/src/components/config/PingTool.tsx
View File

@@ -34,7 +34,7 @@ interface PingStats {
} }
export function PingTool({ tenantId, deviceId }: ConfigPanelProps) { export function PingTool({ tenantId, deviceId }: ConfigPanelProps) {
const [target, setTarget] = useState('') const [target, setTarget] = useState('8.8.8.8')
const [count, setCount] = useState('4') const [count, setCount] = useState('4')
const [size, setSize] = useState('64') const [size, setSize] = useState('64')
const [srcAddress, setSrcAddress] = useState('') const [srcAddress, setSrcAddress] = useState('')

2
frontend/src/components/config/TracerouteTool.tsx
View File

@@ -28,7 +28,7 @@ interface HopResult {
} }
export function TracerouteTool({ tenantId, deviceId }: ConfigPanelProps) { export function TracerouteTool({ tenantId, deviceId }: ConfigPanelProps) {
const [target, setTarget] = useState('') const [target, setTarget] = useState('8.8.8.8')
const [maxHops, setMaxHops] = useState('30') const [maxHops, setMaxHops] = useState('30')
const [timeout, setTimeout] = useState('1000') const [timeout, setTimeout] = useState('1000')
const [protocol, setProtocol] = useState('icmp') const [protocol, setProtocol] = useState('icmp')