Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

fix: vyper contracts re-verificaiton #10053

Merged
merged 4 commits into from
May 14, 2024
Merged

Conversation

rimrakhimov
Copy link
Member

@rimrakhimov rimrakhimov commented May 11, 2024

Motivation

When trying to re-verify Vyper contracts an error with unique constraint violation occurs. Blockscout prints the error, but the verification hangs out indefinitely.

For Solidity contracts re-verification works.

Changelog

Enhancements

Added test cases for publishing the Solidity and Vyper contracts when there is already a verified one.

Bug Fixes

Defined a create_or_update_smart_contract function similar to the one defined for Solidity.Publisher module. Internally, it works the same way: checks if the contract is already verified and based on that calls either the update or create smart-contract method

Checklist for your Pull Request (PR)

@rimrakhimov rimrakhimov force-pushed the rr-fix-vyper-re-verification branch from 87666be to ba6178b Compare May 11, 2024 01:02
@rimrakhimov rimrakhimov requested review from vbaranov, nikitosing, sl1depengwyn and fedor-ivn and removed request for fedor-ivn and sl1depengwyn May 11, 2024 01:07
@nikitosing nikitosing merged commit 621024c into master May 14, 2024
16 checks passed
@nikitosing nikitosing deleted the rr-fix-vyper-re-verification branch May 14, 2024 11:48
fedor-ivn added a commit that referenced this pull request May 18, 2024
* master: (65 commits)
  fix: Add healthcheck endpoints for indexer-only setup (#10076)
  6.6.0
  fix: Rework revert_reason (#9212)
  fix: Eliminate from_address_hash == #{address_hash} clause for transactions query in case of smart-contracts (#9469)
  feat: Add optional retry of NFT metadata fetch in Indexer.Fetcher.Tok… (#10036)
  fix: Separate indexer setup (#10032)
  feat: Blueprint contracts support (#10058)
  chore: Update hackney pool size: add new fetchers accounting (#9941)
  fix: Disallow batched queries in GraphQL endpoint (#10050)
  feat: Clone with immutable arguments proxy pattern (#10039)
  fix: vyper contracts re-verificaiton (#10053)
  refactor: Refactor get_additional_sources/4 -> get_additional_sources/3 (#10046)
  chore: Bump credo from 1.7.5 to 1.7.6 (#10060)
  chore: Bump redix from 1.5.0 to 1.5.1 (#10059)
  chore: Bump ex_doc from 0.32.1 to 0.32.2 (#10061)
  fix: Fix Unknown UID bug at smart-contract verification (#9986)
  refactor: test database config (#9662)
  chore: remove `has_methods` from `/addresses` (#10051)
  feat: Improve retry NFT fetcher (#10027)
  chore: Add support of Blast-specific L1 OP withdrawal events (#10049)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants