chore: increase reason string limit to 1024 bytes

and document it
vyperlang · Oct 25, 2021 · f47906d · f47906d
1 parent 2a4972f
commit f47906d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/docs/statements.rst b/docs/statements.rst
@@ -93,7 +93,7 @@ The ``raise`` statement triggers an exception and reverts the current call.
 
     raise "something went wrong"
 
-The error string is not required.
+The error string is not required. If it is provided, it is limited to 1024 bytes.
 
 assert
 ------
@@ -104,7 +104,7 @@ The ``assert`` statement makes an assertion about a given condition. If the cond
 
     assert x > 5, "value too low"
 
-The error string is not required.
+The error string is not required. If it is provided, it is limited to 1024 bytes.
 
 This method's behavior is equivalent to:
 

diff --git a/vyper/semantics/validation/local.py b/vyper/semantics/validation/local.py
@@ -106,9 +106,9 @@ def _validate_revert_reason(msg_node: vy_ast.VyperNode) -> None:
                 raise StructureException("Reason string cannot be empty", msg_node)
         elif not (isinstance(msg_node, vy_ast.Name) and msg_node.id == "UNREACHABLE"):
             try:
-                validate_expected_type(msg_node, StringDefinition(64))
+                validate_expected_type(msg_node, StringDefinition(1024))
             except TypeMismatch as e:
-                raise InvalidType("revert reason must be String[64]") from e
+                raise InvalidType("revert reason must fit within String[1024]") from e
 
 
 class FunctionNodeVisitor(VyperNodeVisitorBase):