From 994cf9b34b4037780fbd77ff77c472ed71d66492 Mon Sep 17 00:00:00 2001 From: onlineSoftwareDevOK <104783768+onlineSoftwareDevOK@users.noreply.github.com> Date: Mon, 6 Jun 2022 17:27:56 -0400 Subject: [PATCH] Clarify role of MinimalForwarder (#3399) Co-authored-by: Francisco Giordano --- contracts/metatx/MinimalForwarder.sol | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/contracts/metatx/MinimalForwarder.sol b/contracts/metatx/MinimalForwarder.sol index 085f0bb09..8e6e7ee0f 100644 --- a/contracts/metatx/MinimalForwarder.sol +++ b/contracts/metatx/MinimalForwarder.sol @@ -8,6 +8,11 @@ import "../utils/cryptography/draft-EIP712.sol"; /** * @dev Simple minimal forwarder to be used together with an ERC2771 compatible contract. See {ERC2771Context}. + * + * MinimalForwarder is mainly meant for testing, as it is missing features to be a good production-ready forwarder. This + * contract does not intend to have all the properties that are needed for a sound forwarding system. A fully + * functioning forwarding system with good properties requires more complexity. We suggest you look at other projects + * such as the GSN which do have the goal of building a system like that. */ contract MinimalForwarder is EIP712 { using ECDSA for bytes32;