fix on eth_utils to detect address and data from params

CHANGELOG.md

@@ -1,5 +1,4 @@
-## 2.2.0-beta03
-## 2.2.0-beta02
+## 2.2.0-beta04
 
 - Improvements on example wallet
 

example/wallet/lib/dependencies/chains/evm_service.dart

@@ -68,6 +68,7 @@ class EVMService {
 
   Future<void> personalSign(String topic, dynamic parameters) async {
     debugPrint('[$runtimeType] personalSign request: $parameters');
+    // message, address
 
     final pRequest = _web3Wallet.pendingRequests.getAll().first;
     final data = EthUtils.getDataFromParamsList(parameters);
@@ -114,6 +115,7 @@ class EVMService {
 
   Future<void> ethSign(String topic, dynamic parameters) async {
     debugPrint('[$runtimeType] ethSign request: $parameters');
+    // address, message
 
     final pRequest = _web3Wallet.pendingRequests.getAll().first;
     final data = EthUtils.getDataFromParamsList(parameters);
@@ -176,7 +178,7 @@ class EVMService {
         final signature = EthSigUtil.signTypedData(
           privateKey: keys[0].privateKey,
           jsonData: data,
-          version: TypedDataVersion.V1,
+          version: TypedDataVersion.V4,
         );
         response = response.copyWith(result: signature);
       } catch (e) {
@@ -249,6 +251,7 @@ class EVMService {
     );
 
     final data = EthUtils.getTransactionFromParams(parameters);
+    if (data == null) return;
     final result = await approveTransaction(data);
     if (result is Transaction) {
       try {
@@ -301,6 +304,7 @@ class EVMService {
     );
 
     final data = EthUtils.getTransactionFromParams(parameters);
+    if (data == null) return;
     final result = await approveTransaction(data);
     if (result is Transaction) {
       try {

example/wallet/lib/utils/eth_utils.dart

@@ -4,6 +4,11 @@ import 'package:convert/convert.dart';
 import 'package:walletconnect_flutter_v2/walletconnect_flutter_v2.dart';
 
 class EthUtils {
+  static final addressRegEx = RegExp(
+    r'^0x[a-fA-F0-9]{40}$',
+    caseSensitive: false,
+  );
+
   static String getUtf8Message(String maybeHex) {
     if (maybeHex.startsWith('0x')) {
       final List<int> decoded = hex.decode(
@@ -15,26 +20,35 @@ class EthUtils {
     return maybeHex;
   }
 
-  static String getAddressFromParamsList(dynamic params) {
+  static dynamic getAddressFromParamsList(dynamic params) {
     return (params as List).firstWhere((p) {
       try {
-        EthereumAddress.fromHex(p);
-        return true;
+        if (addressRegEx.hasMatch(p)) {
+          EthereumAddress.fromHex(p);
+          return true;
+        }
+        return false;
       } catch (e) {
         return false;
       }
-    });
+    }, orElse: () => null);
   }
 
   static dynamic getDataFromParamsList(dynamic params) {
-    return (params as List).firstWhere((p) {
-      final address = getAddressFromParamsList(params);
-      return p != address;
-    });
+    final address = getAddressFromParamsList(params);
+    final param = (params as List).firstWhere(
+      (p) => p != address,
+      orElse: () => null,
+    );
+    return param;
   }
 
-  static Map<String, dynamic> getTransactionFromParams(dynamic params) {
-    final param = (params as List<dynamic>).first;
-    return param as Map<String, dynamic>;
+  static Map<String, dynamic>? getTransactionFromParams(dynamic params) {
+    final address = getAddressFromParamsList(params);
+    final param = params.firstWhere(
+      (p) => p != address,
+      orElse: () => null,
+    );
+    return param as Map<String, dynamic>?;
   }
 }

pubspec.yaml

@@ -1,6 +1,6 @@
 name: walletconnect_flutter_v2
 description: This repository contains oficial implementation of WalletConnect v2 protocols for Flutter applications. The communications protocol for web3.
-version: 2.2.0-beta03
+version: 2.2.0-beta04
 repository: https://github.com/WalletConnect/WalletConnectFlutterV2
 
 environment: