Revert "Upgrade OZ libraries (#1707)"

This reverts commit cc3b104.

contracts/hardhat.config.ts

@@ -20,7 +20,7 @@ const config: HardhatUserConfig = {
     sources: "src"
   },
   solidity: {
-    version: "0.8.20",
+    version: "0.8.9",
     settings: {
       optimizer: {
         enabled: true,
@@ -55,9 +55,6 @@ const config: HardhatUserConfig = {
   warnings : {
     '*' : {
       default: 'error'
-    },
-    'src/testing/**/*': {
-      default: 'off'
     }
   }
 };

contracts/package.json

@@ -11,18 +11,17 @@
   "license": "ISC",
   "devDependencies": {
     "@nomicfoundation/hardhat-toolbox": "~2.0.0",
+    "@openzeppelin/contracts": "4.5.0",
     "@openzeppelin/hardhat-upgrades": "^1.21.0",
     "@solidstate/hardhat-bytecode-exporter": "^1.1.1",
-    "hardhat": "~2.19.3",
+    "hardhat": "~2.12.4",
     "hardhat-abi-exporter": "^2.10.1",
     "hardhat-deploy": "0.11.42",
     "node-docker-api": "^1.1.22",
     "ts-node": "~10.9.1",
     "typescript": "^4.9.4"
   },
   "dependencies": {
-    "@openzeppelin/contracts": "^5.0.1",
-    "@openzeppelin/contracts-upgradeable": "^5.0.1",
     "hardhat-ignore-warnings": "^0.2.6"
   }
 }

contracts/src/common/ConstantSupplyERC20.sol

@@ -1,13 +1,11 @@
 // SPDX-License-Identifier: Apache 2
 pragma solidity >=0.7.0 <0.9.0;
 
-import {ERC20} from "@openzeppelin/contracts/token/ERC20/ERC20.sol";
-
+import "@openzeppelin/contracts/token/ERC20/ERC20.sol";
 
 contract ConstantSupplyERC20 is ERC20 {
-    constructor(string memory name, string memory symbol, uint256 initialSupply) 
-        ERC20(name, symbol)
-    {
+
+    constructor(string memory name, string memory symbol, uint256 initialSupply) ERC20(name, symbol) {
         _mint(msg.sender, initialSupply);
     }
 }

contracts/src/management/ManagementContract.sol

@@ -2,21 +2,17 @@
 pragma solidity >=0.7.0 <0.9.0;
 
 import "@openzeppelin/contracts/utils/cryptography/ECDSA.sol";
-import "@openzeppelin/contracts-upgradeable/access/OwnableUpgradeable.sol";
-import "@openzeppelin/contracts/utils/cryptography/MessageHashUtils.sol";
+import "@openzeppelin/contracts/access/Ownable.sol";
+import "@openzeppelin/contracts/proxy/utils/Initializable.sol";
 
 
 import "./Structs.sol";
 import * as MessageBus from "../messaging/MessageBus.sol";
 
-contract ManagementContract is Initializable, OwnableUpgradeable {
-
-    using MessageHashUtils for bytes32;
-    using MessageHashUtils for bytes;
+contract ManagementContract is Ownable, Initializable {
 
     constructor() {
-      //  _disableInitializers();
-        _transferOwnership(msg.sender);
+       // _disableInitializers(); //todo @siliev - figure out why the solidity compiler cant find this. Perhaps OZ needs a version upgrade?
     }
 
     event LogManagementContractCreated(address messageBusAddress);
@@ -46,7 +42,6 @@ contract ManagementContract is Initializable, OwnableUpgradeable {
     //The messageBus where messages can be sent to Obscuro
     MessageBus.IMessageBus public messageBus;
     function initialize() public initializer {
-        __Ownable_init(msg.sender);
         lastBatchSeqNo = 0;
         messageBus = new MessageBus.MessageBus();
         emit LogManagementContractCreated(address(messageBus));
@@ -118,16 +113,15 @@ contract ManagementContract is Initializable, OwnableUpgradeable {
         require(isAggAttested);
 
         if (verifyAttester) {
-
             // the data must be signed with by the correct private key
             // signature = f(PubKey, PrivateKey, message)
             // address = f(signature, message)
             // valid if attesterID = address
-            bytes32 calculatedHashSigned = abi.encodePacked(attesterID, requesterID, hostAddress, responseSecret).toEthSignedMessageHash();
+            bytes32 calculatedHashSigned = ECDSA.toEthSignedMessageHash(abi.encodePacked(attesterID, requesterID, hostAddress, responseSecret));
             address recoveredAddrSignedCalculated = ECDSA.recover(calculatedHashSigned, attesterSig);
 
-            require(recoveredAddrSignedCalculated == attesterID, "calculated address and attesterID dont match");
-        } 
+        require(recoveredAddrSignedCalculated == attesterID, "calculated address and attesterID dont match");
+        }
 
         // mark the requesterID aggregator as an attested aggregator and store its host address
         attested[requesterID] = true;

contracts/src/messaging/MessageBus.sol

@@ -7,9 +7,6 @@ import "./Structs.sol";
 import "@openzeppelin/contracts/access/Ownable.sol";
 
 contract MessageBus is IMessageBus, Ownable {
-
-    constructor() Ownable(msg.sender) {}
-
     function messageFee() internal virtual returns (uint256) {
         return 0;
     }

contracts/test/bridge-test.ts

@@ -32,20 +32,10 @@ describe("Bridge", function () {
     const L1Bridge = await hre.ethers.getContractFactory("ObscuroBridge");
     const L2Bridge = await hre.ethers.getContractFactory("EthereumBridge");
 
-    const [owner] = await ethers.getSigners();
-
-    const ERC20 = await hre.ethers.getContractFactory("ConstantSupplyERC20", owner);
-
-    console.log(`Deploying erc20`);
-    try {
-      const erc20 = await ERC20.deploy("XXX", "XXX", 100000);
-      erc20address = erc20.address;
-    } catch(err) {
-      console.error(err);
-    }
-
+    const ERC20 = await hre.ethers.getContractFactory("ERC20");
 
-    console.log(`Deployed erc20`);
+    const erc20 = await ERC20.deploy("XXX", "XXX");
+    erc20address = erc20.address;
 
     busL1 = await MessageBus.deploy();
     busL2 = await MessageBus.deploy();

go/ethadapter/l1_transaction.go

@@ -46,20 +46,17 @@ func (l *L1RespondSecretTx) Sign(privateKey *ecdsa.PrivateKey) *L1RespondSecretT
 	data = append(data, l.HostAddress...)
 	data = append(data, string(l.Secret)...)
 
-	ethereumMessageHash := func(data []byte) []byte {
-		prefix := fmt.Sprintf("\x19Ethereum Signed Message:\n%d", len(data))
-		return crypto.Keccak256([]byte(prefix), data)
-	}
-
-	hashedData := ethereumMessageHash(data)
+	// form the data
+	msg := fmt.Sprintf("\x19Ethereum Signed Message:\n%d%s", len(data), string(data))
+	// hash the data
+	hashedData := crypto.Keccak256Hash([]byte(msg))
 	// sign the hash
-	signedHash, err := crypto.Sign(hashedData, privateKey)
+	signedHash, err := crypto.Sign(hashedData.Bytes(), privateKey)
 	if err != nil {
 		return nil
 	}
-
-	// set recovery id to 27; prevent malleable signatures
-	signedHash[64] += 27
+	// remove ECDSA recovery id
+	signedHash = signedHash[:len(signedHash)-1]
 	l.AttesterSig = signedHash
 	return l
 }

integration/simulation/network/geth_utils.go

@@ -177,40 +177,6 @@ func StopEth2Network(clients []ethadapter.EthClient, netw eth2network.Eth2Networ
 	}
 }
 
-func InitializeContract(workerClient ethadapter.EthClient, w wallet.Wallet, contractAddress common.Address) (*types.Receipt, error) {
-	ctr, err := ManagementContract.NewManagementContract(contractAddress, workerClient.EthClient())
-	if err != nil {
-		return nil, err
-	}
-
-	opts, err := bind.NewKeyedTransactorWithChainID(w.PrivateKey(), w.ChainID())
-	if err != nil {
-		return nil, err
-	}
-
-	tx, err := ctr.Initialize(opts)
-	if err != nil {
-		return nil, err
-	}
-	w.SetNonce(w.GetNonce())
-
-	var start time.Time
-	var receipt *types.Receipt
-	// todo (@matt) these timings should be driven by the L2 batch times and L1 block times
-	for start = time.Now(); time.Since(start) < 80*time.Second; time.Sleep(2 * time.Second) {
-		receipt, err = workerClient.TransactionReceipt(tx.Hash())
-		if err == nil && receipt != nil {
-			if receipt.Status != types.ReceiptStatusSuccessful {
-				return nil, errors.New("unable to initialize contract")
-			}
-			testlog.Logger().Info("Contract initialized")
-			return receipt, nil
-		}
-	}
-
-	return receipt, nil
-}
-
 // DeployContract returns receipt of deployment
 // todo (@matt) - this should live somewhere else
 func DeployContract(workerClient ethadapter.EthClient, w wallet.Wallet, contractBytes []byte) (*types.Receipt, error) {

integration/smartcontract/smartcontracts_test.go

@@ -113,31 +113,12 @@ func TestManagementContract(t *testing.T) {
 			if err != nil {
 				panic(err)
 			}
-
-			nonce, err := client.Nonce(w.Address())
-			if err != nil {
-				t.Error(err)
-			}
-
-			w.SetNonce(nonce)
 			// deploy the same contract to a new address
 			receipt, err := network.DeployContract(client, w, bytecode)
 			if err != nil {
 				t.Error(err)
 			}
 
-			_, err = network.InitializeContract(client, w, receipt.ContractAddress)
-			if err != nil {
-				t.Error(err)
-			}
-
-			nonce, err = client.Nonce(w.Address())
-			if err != nil {
-				t.Error(err)
-			}
-
-			w.SetNonce(nonce)
-
 			// run the test using the new contract, but same wallet
 			test(t,
 				newDebugMgmtContractLib(receipt.ContractAddress, client.EthClient(),