Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(verify): only include optimize infos when single file #1473

Merged

Conversation

mattsse
Copy link
Member

@mattsse mattsse commented May 3, 2022

Motivation

Ref gakonst/ethers-rs#1212

optimization and runs are only applicable in single file mode and not for std json compiler input, because that info is already included in the compiler input object.

Ref https://docs.etherscan.io/api-endpoints/contracts#source-code-submission-gist

Solution

@mattsse mattsse added the Cmd-forge-verify Command: forge verify-contract/check label May 3, 2022
@onbjerg onbjerg added T-feature Type: feature T-bug Type: bug and removed T-feature Type: feature labels May 3, 2022
Copy link
Collaborator

@onbjerg onbjerg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you think this might solve some of the multi-file verification issues, e.g. #1382?

@mattsse
Copy link
Member Author

mattsse commented May 3, 2022

unfortunately no, tested this manually, with various modifications to the payload...

not sure how to patch a hardhat project to log the etherscan request upon verify

the etherscan API responses are also not very helpful -.-

@mattsse mattsse merged commit 2b1bc1e into foundry-rs:master May 3, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Cmd-forge-verify Command: forge verify-contract/check T-bug Type: bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants