Fix proven time (#33)

* chore: add bindings/optimismportal.go

* feat: record onchain proven timestamp

* chore: remove staled line

Makefile

@@ -13,11 +13,17 @@ build-solidity:
 	forge build; \
 	popd;
 
-bindings: build-solidity
+bindings: bindings-L2StandardBridgeBot bindings-OptimismPortal
+
+bindings-L2StandardBridgeBot: build-solidity
 	jq '.abi' contracts/out/L2StandardBridgeBot.sol/L2StandardBridgeBot.json  > contracts/out/L2StandardBridgeBot.sol/L2StandardBridgeBot.abi; \
 	abigen --abi contracts/out/L2StandardBridgeBot.sol/L2StandardBridgeBot.abi --pkg bindings --type L2StandardBridgeBot --out bindings/L2StandardBridgeBot.go
 
+bindings-OptimismPortal:
+	curl -o bindings/OptimismPortal.go https://raw.githubusercontent.com/ethereum-optimism/optimism/005be54bde97747b6f1669030721cd4e0c14bc69/op-bindings/bindings/optimismportal.go
+
 .PHONY: \
 	bot  \
 	build-go \
-	build-solidity
+	build-solidity \
+	bindings

bindings/README.md

@@ -0,0 +1,2 @@
+
+**Note that all files under this directory are autogenerated via `make bindings`. For details, please refer to ../Makefile.**

cmd/bot/run.go

@@ -144,12 +144,16 @@ func ProcessUnprovenBotDelegatedWithdrawals(ctx context.Context, log log.Logger,
 			continue
 		}
 
-		now := time.Now()
 		err := processor.ProveWithdrawalTransaction(ctx, &unproven, *currentNonce)
 		if err != nil {
 			if strings.Contains(err.Error(), "OptimismPortal: withdrawal hash has already been proven") {
 				// The withdrawal has already proven, mark it
-				result := db.Model(&unproven).Update("proven_time", now)
+				provenTime, err := processor.GetProvenTime(&unproven)
+				if err != nil {
+					log.Error("fail to get proven time", "error", err)
+				}
+
+				result := db.Model(&unproven).Update("proven_time", provenTime)
 				if result.Error != nil {
 					log.Error("failed to update proven withdrawals", "error", result.Error)
 				}

core/processor.go

@@ -242,6 +242,35 @@ func (b *Processor) FinalizeMessage(ctx context.Context, wi *WithdrawalInitiated
 	return nil
 }
 
+func (b *Processor) GetProvenTime(wi *WithdrawalInitiatedLog) (*big.Int, error) {
+	optimismPortal, err := bindings.NewOptimismPortalCaller(b.cfg.L1Contracts.OptimismPortalProxy, b.L1Client)
+	if err != nil {
+		return nil, err
+	}
+
+	receipt, err := b.L1Client.TransactionReceipt(context.Background(), common.HexToHash(wi.TransactionHash))
+	if err != nil {
+		return nil, err
+	}
+
+	withdrawal, err := b.toWithdrawal(wi, receipt)
+	if err != nil {
+		return nil, err
+	}
+
+	withdrawalHash, err := b.hashWithdrawal(withdrawal)
+	if err != nil {
+		return nil, err
+	}
+
+	provenWithdrawal, err := optimismPortal.ProvenWithdrawals(nil, common.HexToHash(withdrawalHash))
+	if err != nil {
+		return nil, err
+	}
+
+	return provenWithdrawal.Timestamp, nil
+}
+
 func (b *Processor) hashWithdrawal(w *bindings.TypesWithdrawalTransaction) (string, error) {
 	uint256Type, _ := abi.NewType("uint256", "", nil)
 	addressType, _ := abi.NewType("address", "", nil)