Skip to content
This repository was archived by the owner on Jan 10, 2026. It is now read-only.
This repository was archived by the owner on Jan 10, 2026. It is now read-only.

Support tokens with callback hooks #219

Description

@fubuloubu

From audit:

"Callbacks. Some token standards, e.g. ERC223, ERC721, and ERC777, define callbacks. Some of these callbacks can be invoked upon regular “transfer” calls. This creates a significant security risk in the form of reentrancies. Inside the strategy additional protection measures might be required depending on the token type and its handling. The vault already has reentrancy locks that should be sufficient for most tokens."

NOTE: Currently we do not support other token standards besides ERC20, so it remains to be seen if we would like to support these.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or requestp3Fix if time

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions