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

Enable writing to the verifier alliance test database in staging #1719

Open
wants to merge 15 commits into
base: staging
Choose a base branch
from

Conversation

marcocastignoli
Copy link
Member

@marcocastignoli marcocastignoli commented Oct 28, 2024

This PR simply enables the WStorageIdentifiers.AllianceDatabase in the staging configuration under writeOrWarn storage services.

marcocastignoli and others added 15 commits October 14, 2024 14:55
* Refactor database utils into class

* add commented  SOURCIFY_POSTGRES_SCHEMA, ALLIANCE_POSTGRES_SCHEMA in .env.dev

* add custom schema comment default public
add metis mainnet and testnet
* Added Zircuit Testnet and Mainnet

* Zircuit update
* Enabling Flare Mainnet Chain-14 support

* Fixing issue

* Fixing issue
…CE2dF4C69aA809De5b9318a3D4452D86BE to partial (#1705)
* Add generic APIKey support in chain RPCs, instead of Alchemy Infura etc.

* Mark rpcs with trace_transaction support

* Throw when API key not found, add QUICKNODE_API_KEY env

* Fix typo in RPC type

* Fix extracting wrong fields from the traces

* Fix checking address created by the creatorTxHash for create2

For contracts created with create2 and factories, the old way of checking the address being created with tx.from and tx.nonce does not work.

Instead we can use the already provided address in the trace. For EOA created ones, we can use the one found in the txReceipt.

* Fix build due to unused function

* Add tests for traces, install sinon et. al. to lib-sourcify

* Add chains that have `trace_transaction` support in Quicknode

* Add creationbytecode extractor for `debug_traceTransaction` traces.

* Add chains with trace `debug_traceTransaction", add Arbitrum Nova Etherscan

* Fix Alchemy type chain

* Show traceSupportedRPCs in /chains response. Don't show API Keys in the response

* Fix type error `findCreateInDebugTraceTransactionCalls` not being copied over to `testChains`

* Add comments to `traceSupportedRPCs`

* Add quicknode subdomains as option

* Fix typing and env handling

* Fix typo
The changes adds the sourcify-server package to be published to npm.

It also adds a check to only publish the packages in their own tagged CI runs. Previously e.g.  when there was a new tag for theese packages, both lib-sourcify and bytecode-utils's CI runs tried to publish both packages. This results in the first one succeeding and publishing both packages but the second one to fail.

Now we check to only publish the package in their corresponding tag CI builds.
@manuelwedler
Copy link
Collaborator

This needs a rebase after rewriting the staging history

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Sprint - Needs Review
Development

Successfully merging this pull request may close these issues.

7 participants