fix(l1): update existing contact ENR on NODES response

Author: ElFantasma

crates/networking/p2p/discv5/peer_table.rs

@@ -843,28 +843,45 @@ impl PeerTableServer {
 
     async fn new_contact_records(&mut self, node_records: Vec<NodeRecord>, local_node_id: H256) {
         for node_record in node_records {
+            if !node_record.verify_signature() {
+                continue;
+            }
             if let Ok(node) = Node::from_enr(&node_record) {
                 let node_id = node.node_id();
-                if let Entry::Vacant(vacant_entry) = self.contacts.entry(node_id)
-                    && !self.discarded_contacts.contains(&node_id)
-                    && node_id != local_node_id
-                {
-                    let mut contact = Contact::from(node);
-                    let is_fork_id_valid =
-                        if let Some(remote_fork_id) = node_record.decode_pairs().eth {
-                            backend::is_fork_id_valid(&self.store, &remote_fork_id)
-                                .await
-                                .ok()
-                                .or(Some(false))
-                        } else {
-                            Some(false)
-                        };
-                    contact.is_fork_id_valid = is_fork_id_valid;
-                    contact.record = Some(node_record);
-                    vacant_entry.insert(contact);
-                    METRICS.record_new_discovery().await;
+                if self.discarded_contacts.contains(&node_id) || node_id == local_node_id {
+                    continue;
+                }
+                let is_fork_id_valid =
+                    if let Some(remote_fork_id) = node_record.decode_pairs().eth {
+                        backend::is_fork_id_valid(&self.store, &remote_fork_id)
+                            .await
+                            .ok()
+                            .or(Some(false))
+                    } else {
+                        Some(false)
+                    };
+                match self.contacts.entry(node_id) {
+                    Entry::Vacant(vacant_entry) => {
+                        let mut contact = Contact::from(node);
+                        contact.is_fork_id_valid = is_fork_id_valid;
+                        contact.record = Some(node_record);
+                        vacant_entry.insert(contact);
+                        METRICS.record_new_discovery().await;
+                    }
+                    Entry::Occupied(mut occupied_entry) => {
+                        let existing_seq = occupied_entry
+                            .get()
+                            .record
+                            .as_ref()
+                            .map_or(0, |r| r.seq);
+                        if node_record.seq > existing_seq {
+                            let contact = occupied_entry.get_mut();
+                            contact.node = node;
+                            contact.record = Some(node_record);
+                            contact.is_fork_id_valid = is_fork_id_valid;
+                        }
+                    }
                 }
-                // TODO Handle the case the contact is already present
             }
         }
     }