From bc7011022fa3e10bb7b53a0b1d9a33727b85b955 Mon Sep 17 00:00:00 2001
From: AnshuJalan <anshujalan206@gmail.com>
Date: Thu, 19 Sep 2024 14:07:30 +0530
Subject: [PATCH] feat: comment out bls checks

---
 SmartContracts/src/avs/PreconfRegistry.sol | 38 ++++++++++++----------
 1 file changed, 21 insertions(+), 17 deletions(-)

diff --git a/SmartContracts/src/avs/PreconfRegistry.sol b/SmartContracts/src/avs/PreconfRegistry.sol
index d5de8e6..ef15a53 100644
--- a/SmartContracts/src/avs/PreconfRegistry.sol
+++ b/SmartContracts/src/avs/PreconfRegistry.sol
@@ -105,17 +105,19 @@ contract PreconfRegistry is IPreconfRegistry, BLSSignatureChecker, Initializable
                 revert PreconferNotRegistered();
             }
 
-            bytes memory message = _createMessage(ValidatorOp.ADD, addValidatorParams[i].signatureExpiry, msg.sender);
+            // Note: BLS signature checks are commented out for the POC
+
+            // bytes memory message = _createMessage(ValidatorOp.ADD, addValidatorParams[i].signatureExpiry, msg.sender);
 
             // Revert if any signature is invalid
-            if (!verifySignature(message, addValidatorParams[i].signature, addValidatorParams[i].pubkey)) {
-                revert InvalidValidatorSignature();
-            }
+            // if (!verifySignature(message, addValidatorParams[i].signature, addValidatorParams[i].pubkey)) {
+            //     revert InvalidValidatorSignature();
+            // }
 
             // Revert if the signature has expired
-            if (block.timestamp > addValidatorParams[i].signatureExpiry) {
-                revert ValidatorSignatureExpired();
-            }
+            // if (block.timestamp > addValidatorParams[i].signatureExpiry) {
+            //     revert ValidatorSignatureExpired();
+            // }
 
             // Point compress the public key just how it is done on the consensus layer
             uint256[2] memory compressedPubKey = addValidatorParams[i].pubkey.compress();
@@ -167,18 +169,20 @@ contract PreconfRegistry is IPreconfRegistry, BLSSignatureChecker, Initializable
                 revert ValidatorAlreadyInactive();
             }
 
-            bytes memory message =
-                _createMessage(ValidatorOp.REMOVE, removeValidatorParams[i].signatureExpiry, validator.preconfer);
+            // Note: BLS signature checks have been commented out
 
-            // Revert if any signature is invalid
-            if (!verifySignature(message, removeValidatorParams[i].signature, removeValidatorParams[i].pubkey)) {
-                revert InvalidValidatorSignature();
-            }
+            // bytes memory message =
+            //     _createMessage(ValidatorOp.REMOVE, removeValidatorParams[i].signatureExpiry, validator.preconfer);
 
-            // Revert if the signature has expired
-            if (block.timestamp > removeValidatorParams[i].signatureExpiry) {
-                revert ValidatorSignatureExpired();
-            }
+            // // Revert if any signature is invalid
+            // if (!verifySignature(message, removeValidatorParams[i].signature, removeValidatorParams[i].pubkey)) {
+            //     revert InvalidValidatorSignature();
+            // }
+
+            // // Revert if the signature has expired
+            // if (block.timestamp > removeValidatorParams[i].signatureExpiry) {
+            //     revert ValidatorSignatureExpired();
+            // }
 
             unchecked {
                 // We also need to delay the removal by two epochs to avoid contradicting the lookahead