Refactor NetworkNode and NetworkView components to streamline node selection logic and remove unused well-known ports

Author: rajsinghtech

frontend/src/components/NetworkNode.tsx

@@ -37,14 +37,6 @@ const WELL_KNOWN_PORTS: Record<number, string> = {
   53: 'DNS',
   80: 'HTTP',
   443: 'HTTPS',
-  3389: 'RDP',
-  5432: 'PostgreSQL',
-  3306: 'MySQL',
-  6379: 'Redis',
-  8080: 'HTTP-Alt',
-  8443: 'HTTPS-Alt',
-  9090: 'Prometheus',
-  27017: 'MongoDB',
 };
 
 const NetworkNode = memo<NodeProps>(({ data, selected }) => {

frontend/src/pages/NetworkView.tsx

@@ -1424,18 +1424,44 @@ const NetworkView: React.FC = () => {
           setSelectedLink(null)
         }}
         onSelectLog={(logEntry) => {
-          // Highlight the corresponding nodes in the graph
-          const srcNode = nodes.find(n => 
+          // First, find the source and destination nodes
+          const srcNode = nodes.find(n =>
             n.ips?.includes(logEntry.srcIP) || n.ip === logEntry.srcIP || n.displayName === logEntry.srcDevice
           )
-          const dstNode = nodes.find(n => 
+          const dstNode = nodes.find(n =>
             n.ips?.includes(logEntry.dstIP) || n.ip === logEntry.dstIP || n.displayName === logEntry.dstDevice
           )
-          
-          if (srcNode) {
+
+          // Try to find the corresponding link/edge between these nodes
+          if (srcNode && dstNode) {
+            const matchingLink = links.find(link => {
+              const linkSourceId = typeof link.source === 'string' ? link.source : link.source.id
+              const linkTargetId = typeof link.target === 'string' ? link.target : link.target.id
+
+              // Check both directions (src->dst or dst->src)
+              return (
+                (linkSourceId === srcNode.id && linkTargetId === dstNode.id) ||
+                (linkSourceId === dstNode.id && linkTargetId === srcNode.id)
+              )
+            })
+
+            if (matchingLink) {
+              // Select the link/edge instead of a node
+              setSelectedLink(matchingLink)
+              setSelectedNode(null)
+            } else {
+              // If no link found, fall back to selecting the source node
+              setSelectedNode(srcNode)
+              setSelectedLink(null)
+            }
+          } else if (srcNode) {
+            // Only source node found
             setSelectedNode(srcNode)
+            setSelectedLink(null)
           } else if (dstNode) {
+            // Only destination node found
             setSelectedNode(dstNode)
+            setSelectedLink(null)
           }
         }}
       />