+
Skip to content

Feat: Sync for ETH settlement in CurrencySettler.sol #958

@z0r0z

Description

@z0r0z

Component

No response

Describe the suggested feature and problem it solves.

The test util, CurrencySettler.sol currently does not sync() for the ETH case.

From PoolManager.sol, "if settling native, integrators should still call sync first to avoid DoS attack vectors".

file: https://github.com/Uniswap/v4-core/blob/main/test/utils/CurrencySettler.sol

ht @kadenzipfel for pointing this out in review of the community V4 swap router.

Describe the desired implementation.

No response

Describe alternatives.

No response

Additional context.

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载